The patch titled Subject: drivers/video/backlight/lm3639_bl.c: fix up world writable sysfs file has been added to the -mm tree. Its filename is drivers-video-backlight-lm3639_blc-fix-up-world-writable-sysfs-file.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Axel Lin <axel.lin@xxxxxxxxxx> Subject: drivers/video/backlight/lm3639_bl.c: fix up world writable sysfs file We don't need the sysfs file to be world writable or group writable. This file is write-only, change it to S_IWUSR (0200). Signed-off-by: Axel Lin <axel.lin@xxxxxxxxxx> Acked-by: G.Shark Jeong<gshark.jeong@xxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/video/backlight/lm3639_bl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/video/backlight/lm3639_bl.c~drivers-video-backlight-lm3639_blc-fix-up-world-writable-sysfs-file drivers/video/backlight/lm3639_bl.c --- a/drivers/video/backlight/lm3639_bl.c~drivers-video-backlight-lm3639_blc-fix-up-world-writable-sysfs-file +++ a/drivers/video/backlight/lm3639_bl.c @@ -214,7 +214,7 @@ out_input: } -static DEVICE_ATTR(bled_mode, 0666, NULL, lm3639_bled_mode_store); +static DEVICE_ATTR(bled_mode, S_IWUSR, NULL, lm3639_bled_mode_store); /* torch */ static void lm3639_torch_brightness_set(struct led_classdev *cdev, _ Patches currently in -mm which might be from axel.lin@xxxxxxxxxx are linux-next.patch drivers-video-backlight-lm3639_blc-fix-up-world-writable-sysfs-file.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html