Hi!

>BOOL copy(COUNT drive, BYTE * srcPath, BYTE * rootPath, BYTE * file)
[...]
>  if (!check_space(drive, filelength(fdin)))
>  {
>    printf("%s: Not enough space to transfer %s\n", pgm, file);
>    close(fdin);
>    exit(1);
>  }
[...]
>    if (write(fdout, copybuffer, ret) != ret)
>    {
>      printf("Can't write %u bytes to %s\n", ret, dest);
>      close(fdout);
>      unlink(dest);
>      break;
>    }
[...]
>    _dos_setftime(fdout, date, time);
[...]
>  close(fdout);

- I think, checking for space may be ommited (because later checked if
  writing is successful).

- `break' after writing error looks like bug, because later to fdout applied
  setftime() and close(). Instead `break' should be applied `return FALSE'
  or `exit(1)' (as with check_space()).




-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
_______________________________________________
Freedos-kernel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/freedos-kernel

Reply via email to