Messages in this thread Patch in this message | | | Date | Wed, 23 Oct 2002 20:53:36 -0700 | From | "Randy.Dunlap" <> | Subject | [trivial] module.c double init? |
| |
in 2.5.44: kernel/module.c, function s_start(), there is: loff_t n = *pos;
and then same "n = *pos" is done in the for-loop initializer. It's just a thinko, right? or am I missing something?
Patch attached (to 2.5.44).
~Randy--- ./kernel/module.c.fixit Fri Oct 18 21:01:17 2002 +++ ./kernel/module.c Wed Oct 23 19:54:10 2002 @@ -1165,7 +1165,7 @@ if (!p) return ERR_PTR(-ENOMEM); lock_kernel(); - for (v = module_list, n = *pos; v; n -= v->nsyms, v = v->next) { + for (v = module_list; v; n -= v->nsyms, v = v->next) { if (n < v->nsyms) { p->mod = v; p->index = n; | |