On Saturday 13 September 2008 22:09, [EMAIL PROTECTED] wrote:
> The replacement to modprobe-small.
> I explained the changes in "modprobe patches" mailing list thread an hour ago.
> "speeds" contains profiling info for the 3 flavors: fast, small and Timo s.
> A typical speedup is of up to 10 times.
> 
> Please, comment and consider applying.

+       size_t len;
...
+               // goto next module definition
+               for (int i = 3; --i >= 0; ) {
+                       ptr += len;
+                       len = *((unsigned *)ptr);
+                       ptr += sizeof(len);
+               }

sizeof(size_t) may be != 4. sizeof(unsigned) also may be != 4.
Also, this may perform illegal unaligned accesses.

+       write(STDOUT_FILENO, &stringbuf_idx, 4);

This is wrong for the same reasons.
--
vda
_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to