lkml.org 
[lkml]   [2011]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[113/262] carminefb: Fix module parameters permissions
    3.0-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Jean Delvare <jdelvare@suse.de>

    commit c84c14224bbca6ec60d5851fcc87be0e34df2f44 upstream.

    The third parameter of module_param is supposed to be an octal value.
    The missing leading "0" causes the following:

    $ ls -l /sys/module/carminefb/parameters/
    total 0
    -rw-rwxr-- 1 root root 4096 Jul 8 08:55 fb_displays
    -rw-rwxr-- 1 root root 4096 Jul 8 08:55 fb_mode
    -rw-rwxr-- 1 root root 4096 Jul 8 08:55 fb_mode_str

    After fixing the perm parameter, we get the expected:

    $ ls -l /sys/module/carminefb/parameters/
    total 0
    -r--r--r-- 1 root root 4096 Jul 8 08:56 fb_displays
    -r--r--r-- 1 root root 4096 Jul 8 08:56 fb_mode
    -r--r--r-- 1 root root 4096 Jul 8 08:56 fb_mode_str

    Signed-off-by: Jean Delvare <jdelvare@suse.de>
    Cc: Paul Mundt <lethal@linux-sh.org>
    Cc: Sebastian Siewior <bigeasy@linutronix.de>
    Signed-off-by: Paul Mundt <lethal@linux-sh.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/video/carminefb.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    --- a/drivers/video/carminefb.c
    +++ b/drivers/video/carminefb.c
    @@ -32,11 +32,11 @@
    #define CARMINEFB_DEFAULT_VIDEO_MODE 1

    static unsigned int fb_mode = CARMINEFB_DEFAULT_VIDEO_MODE;
    -module_param(fb_mode, uint, 444);
    +module_param(fb_mode, uint, 0444);
    MODULE_PARM_DESC(fb_mode, "Initial video mode as integer.");

    static char *fb_mode_str;
    -module_param(fb_mode_str, charp, 444);
    +module_param(fb_mode_str, charp, 0444);
    MODULE_PARM_DESC(fb_mode_str, "Initial video mode in characters.");

    /*
    @@ -46,7 +46,7 @@ MODULE_PARM_DESC(fb_mode_str, "Initial v
    * 0b010 Display 1
    */
    static int fb_displays = CARMINE_USE_DISPLAY0 | CARMINE_USE_DISPLAY1;
    -module_param(fb_displays, int, 444);
    +module_param(fb_displays, int, 0444);
    MODULE_PARM_DESC(fb_displays, "Bit mode, which displays are used");

    struct carmine_hw {



    \
     
     \ /
      Last update: 2011-11-10 04:37    [W:4.148 / U:0.384 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site