Re: [patch] 2.4.18/2.5.24 kernel/module.c - minor bugs

From: Keith Owens (kaos@ocs.com.au)
Date: Mon Jul 08 2002 - 05:59:37 EST


On Mon, 8 Jul 2002 10:27:50 +0200,
Peter Oberparleiter <oberpapr@softhome.net> wrote:
>this patch fixes two minor bugs in kernel/module.c in current linux
>kernel versions (2.4.18/2.5.24) which could cause problems in some
>rare situations:
>1. A size-check in sys_create_module is off by one. The check reads
>2. In case "struct module" used by insmod is larger than the one used
>by the kernel (e.g. newer version), module loading will fail.

Looks good. Linus/Marcelo, please apply.

========================================
--- linux-2.4.17/kernel/module.c Sun Nov 11 20:23:14 2001
+++ linux-2.4.17-modfix/kernel/module.c Mon Jul 8 09:50:57 2002
@@ -303,7 +303,7 @@
                 error = namelen;
                 goto err0;
         }
- if (size < sizeof(struct module)+namelen) {
+ if (size < sizeof(struct module)+namelen+1) {
                 error = -EINVAL;
                 goto err1;
         }
@@ -482,10 +482,10 @@
                 error = n_namelen;
                 goto err2;
         }
- if (namelen != n_namelen || strcmp(n_name, mod_tmp.name) != 0) {
+ if (namelen != n_namelen || strcmp(n_name, name_tmp) != 0) {
                 printk(KERN_ERR "init_module: changed module name to "
                                 "`%s' from `%s'\n",
- n_name, mod_tmp.name);
+ n_name, name_tmp);
                 goto err3;
         }
 
========================================

>Regards,
> Peter Oberparleiter

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Jul 15 2002 - 22:00:12 EST