Hello,

>         if (!(fp = fopen (im->real_file, "rb"))) {
> +               close (dest);
>                 unlink (tmp);

    I guess the fopen might fail if the user hasn't got permission to
    read the file. The fix looks good, although for performance it may
    be useful to check the file starts with "BZ" before even trying to
    create a temporary file.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to