[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