[LEDE-DEV] [PATCH ubox 3/6] kmodloader: modprobe: skip possible command line arguments

Felix Fietkau nbd at nbd.name
Fri Jan 13 01:02:18 PST 2017


On 2017-01-11 12:54, Yousong Zhou wrote:
> The kernel may invocate user mode modprobe with the following scheme
> 
>     modprobe -q -- <module_name>
> 
> Signed-off-by: Yousong Zhou <yszhou4tech at gmail.com>
> ---
>  kmodloader.c | 13 +++++++++++--
>  1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/kmodloader.c b/kmodloader.c
> index b20de6e..b5dc4d1 100644
> --- a/kmodloader.c
> +++ b/kmodloader.c
> @@ -719,8 +719,17 @@ static int main_modprobe(int argc, char **argv)
>  {
>  	struct module *m;
>  	char *name;
> +	char *mod = NULL;
> +	int i;
>  
> -	if (argc != 2)
> +	if (argc < 2)
> +		return print_usage("modprobe");
> +	for (i = 1; i < argc; i++)
> +		if (argv[i][0] != '-') {
Wouldn't it make more sense to use getopt here and fail on invalid options?

- Felix




More information about the Lede-dev mailing list