net: usb: raw_ip: max_intfs to be non-writable
JC Kuo [Tue, 13 Nov 2012 06:36:03 +0000 (14:36 +0800)]
max_intfs is a varible which determines loop counts in driver
functions at runtime. Userspace could modify its default value at
module load time but it shouldn't be changed at runtime.

Bug 1175543

Change-Id: Id8e8055711288f900e5cffb704a2cf823dd97a34
Signed-off-by: JC Kuo <jckuo@nvidia.com>
Reviewed-on: http://git-master/r/187794
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>

drivers/net/usb/raw_ip_net.c

index bebf406..8790a67 100644 (file)
@@ -59,7 +59,8 @@ static unsigned long usb_net_raw_ip_intf[MAX_INTFS] = { 3, 5, 9, 11, 13};
 unsigned long usb_net_raw_ip_rx_debug;
 unsigned long usb_net_raw_ip_tx_debug;
 
-module_param(max_intfs, int, 0644);
+/* max_intfs should not be changed at runtime */
+module_param(max_intfs, int, S_IRUGO);
 MODULE_PARM_DESC(max_intfs, "usb net (raw-ip) - max. interfaces supported");
 module_param(usb_net_raw_ip_rx_debug, ulong, 0644);
 MODULE_PARM_DESC(usb_net_raw_ip_rx_debug, "usb net (raw-ip) - rx debug");
@@ -1031,7 +1032,7 @@ static int usb_net_raw_ip_init(void)
        int err;
        char name[32];
 
-       pr_debug("usb_net_raw_ip_init {\n");
+       pr_debug("usb_net_raw_ip_init { max_intfs %d\n", max_intfs);
 
        err = usb_register(&baseband_usb_driver);
        if (err < 0) {