Re: [PATCH] Smackv10: Smack rules grammar + their stateful parser

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

 



On Tue, Nov 06, 2007 at 03:08:08PM -0800, Linus Torvalds wrote:
> 
> 
> On Wed, 7 Nov 2007, Adrian Bunk wrote:
> > 
> > You were the one who suggested to _parse_ strings in the kernel.
> 
> So?
> 
> We do that for lots of things.
> 
> What do you think a filename is? And yes, we parse it. Things like '/' and 
> '.' and '..' have magic meaning.
> 
> You don't need to bring up idiotic things like character sets. You can see 
> it as a byte string. You're done with it.

We have the following properties in the character sets we handle:
- every ASCII character is encoded with the same byte as in ASCII
- if the eighth bit is 0, the byte can't be part of a multi-byte character
- no ASCII character can be encoded in a different way

This (plus most likely some other properties I've missed to mention) 
allows some parsing based on ASCII characters.

But if you want to match "one character" (like TOMOYO does) or want to 
check for printable characters except space (like Smack does) you must 
know whether the byte string 0xC3 0xA0 is the character à or a sequence 
of two characters with the second one being NBSP.

> 		Linus

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
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