Re: [PATCH] Add spaces on either side of case "..." operator.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 10 Apr 2007 07:23:35 -0400 (EDT) "Robert P. J. Day" <[email protected]> wrote:

> Following the programming advice laid down in the gcc manual, make
> sure the case "..." operator has spaces on either side.
> 
> Signed-off-by: Robert P. J. Day <[email protected]>
> 
> ---
> 
> According to:
> 
> http://gcc.gnu.org/onlinedocs/gcc-4.1.2/gcc/Case-Ranges.html#Case-Ranges:
> 
>   "Be careful: Write spaces around the ..., for otherwise it may be
> parsed wrong when you use it with integer values."
> 
> 
> diff --git a/kernel/audit.c b/kernel/audit.c
> index 76c9a11..d963a04 100644
> --- a/kernel/audit.c
> +++ b/kernel/audit.c
> @@ -515,8 +515,8 @@ static int audit_netlink_ok(struct sk_buff *skb, u16 msg_type)
>  			err = -EPERM;
>  		break;
>  	case AUDIT_USER:
> -	case AUDIT_FIRST_USER_MSG...AUDIT_LAST_USER_MSG:
> -	case AUDIT_FIRST_USER_MSG2...AUDIT_LAST_USER_MSG2:
> +	case AUDIT_FIRST_USER_MSG ... AUDIT_LAST_USER_MSG:
> +	case AUDIT_FIRST_USER_MSG2 ... AUDIT_LAST_USER_MSG2:

hm... We have: 

#define AUDIT_FIRST_USER_MSG    1100    /* Userspace messages mostly uninteresting to kernel */
#define AUDIT_USER_AVC          1107    /* We filter this differently */
#define AUDIT_LAST_USER_MSG     1199

and CPP turns that into

 case 1100 ...1199:
 case 2100 ...2999:

and it does the same when the comments are stripped from the #defines.

So we were saved by the trailing space which cpp added to the expanded
macro.  I wonder why cpp did that, and to what extent one can rely cpp
doing that.

Oh well.  I guess we should apply the patch.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux