[U-Boot] fs/fs.c - error handling needed?

Dear Simon,

with commit a8f6ab5 "fs: Add support for saving data to filesystems" you add the function do_save() to U-Boot. This includes the following code (line numbers as of current master):

"fs/fs.c":

... 331 filename = argv[3]; 332 addr = simple_strtoul(argv[4], NULL, cmdline_base); 333 bytes = simple_strtoul(argv[5], NULL, cmdline_base); 334 if (argc >= 7) 335 pos = simple_strtoul(argv[6], NULL, cmdline_base); 336 else 337 pos = 0;

Should we not perform at least minimal error checking, i. e. verify that no garbage arguments have been passed to that function?

Best regards, Viele Grüße,

Wolfgang Denk