74d443bb2d
A comment about a pre-existing bug data structure definition is added.
This bug was evidently introduced by Xiong Zhou in the patch
bd471258f2
("staging: android: logger: use kuid_t instead of uid_t")
where the code was changed to hide a valid build warning instead of
solving the bug that was identified by it.
Signed-off-by: Somya Anand <somyaanand214@gmail.com>
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
18 lines
658 B
Plaintext
18 lines
658 B
Plaintext
TODO:
|
|
- checkpatch.pl cleanups
|
|
- sparse fixes
|
|
- rename files to be not so "generic"
|
|
- make sure things build as modules properly
|
|
- add proper arch dependencies as needed
|
|
- audit userspace interfaces to make sure they are sane
|
|
- kuid_t should never be exposed to user space as it is
|
|
kernel internal type. Data structure for this kuid_t is:
|
|
typedef struct {
|
|
uid_t val;
|
|
} kuid_t;
|
|
- This bug is introduced by Xiong Zhou in the patch bd471258f2e09
|
|
- ("staging: android: logger: use kuid_t instead of uid_t")
|
|
|
|
Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc:
|
|
Brian Swetland <swetland@google.com>
|