[U-Boot-Users] patch to tftp.c

this patch for "tftp.c" fixes a logical error in the prior implementation supporting direct-tftp-to-flash. this puts in some additional safety checks to make sure you don't cross a flash boundary during the course of a write.
this will catch such actions before starting, and complain verbosely about what the actual problem is.
-josh

In message 20040221004708.01cbfbda.fryman@cc.gatech.edu you wrote:
this patch for "tftp.c" fixes a logical error in the prior implementation supporting direct-tftp-to-flash. this puts in some additional safety checks to make sure you don't cross a flash boundary during the course of a write.
this will catch such actions before starting, and complain verbosely about what the actual problem is.
How can you know that in advance? There is no indication how big the file will become.
I don't really see which problem you are trying to fix (especially since downloading directlty to flash is inherently insecure and discouraged).
There is again no CHANGELOG entry.
Again there is violation of coding standards (trailing white space; brace style).
Rejected.
Best regards,
Wolfgang Denk
participants (2)
-
Josh Fryman
-
Wolfgang Denk