Re: [PATCH 13/13] docs: update TODO

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

 



On Sunday 14 October 2012 16:22:25 Sami Kerola wrote:
> - - (!) rewrite ipcs to use /proc/sys/kernel rather than unreliable
> syscalls
> -   (there are problems with 32bit userspace on 64bit kernel)

this makes me a bit suspicious since the new code turns around and uses size_t 
to read values.  that says to me that the same problems that [allegedly] 
plague the kernel syscall interface will bite your proc reading code.

looking at the most common case (which will directly apply to all the others):
 - 32bit ia32 kernel: sizeof(size_t) == 4
 - 64bit x86_64 kernel: sizeof(size_t) == 8
  - 64bit x86_64 userland: sizeof(size_t) == 8
  - 32bit ia32 userland: sizeof(size_t) == 4
  - 32bit x86_x32 userland: sizeof(size_t) == 4

so running any of the last two setups could (in theory) be problematic because 
this new code still uses size_t.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux