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