diff options
| author | Andrew Morton <akpm@osdl.org> | 2006-02-01 12:05:02 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-02-01 17:53:12 +0100 |
| commit | 1c40f7d4f0a9d5242f19b02b00e3e5a8ee218a20 (patch) | |
| tree | 3a6422d574bf033ef1c4c6f455c6e528ac3116c2 /mm | |
| parent | [PATCH] tpm_bios: securityfs error checking fix (diff) | |
| download | linux-1c40f7d4f0a9d5242f19b02b00e3e5a8ee218a20.tar.xz linux-1c40f7d4f0a9d5242f19b02b00e3e5a8ee218a20.zip | |
[PATCH] tpm_bios indexing fix
It generates warnings:
drivers/char/tpm/tpm_bios.c: In function `get_event_name':
drivers/char/tpm/tpm_bios.c:223: warning: cast from pointer to integer of different size
drivers/char/tpm/tpm_bios.c:223: warning: cast from pointer to integer of different size
drivers/char/tpm/tpm_bios.c:223: warning: cast from pointer to integer of different size
drivers/char/tpm/tpm_bios.c:224: warning: cast from pointer to integer of different size
drivers/char/tpm/tpm_bios.c:224: warning: cast from pointer to integer of different size
drivers/char/tpm/tpm_bios.c:224: warning: cast from pointer to integer of different size
and I'm not sure what the code is doing there, but it seems wrong. We're
using the address of the buffer rather than the contents of it.
The patch adds more nasty typecasting, but I think the whole arrangement could
be done in a more typesafe manner.
Cc: Kylene Jo Hall <kjhall@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions