Re: [Wireshark-commits] rev 39318: /trunk/epan/ /trunk/epan/: filesystem.c

[Stig Bjørlykke <stig () bjorlykke org>]

On Sun, Oct 9, 2011 at 6:28 PM, Guy Harris <guy () alum mit edu> wrote:
> Code that calls file_exists() should have detected the null pointer before that, and reported the appropriate error 
> to the user or chosen a default file name or whatever would be appropriate in that case; what code was passing a null 
> pointer to file_exists()?

I found that file_exists() returned true for a NULL pointer, so i
added the test for NULL while adding some changes in
uat_get_actual_filename().  I could have checked the file name before
calling file_exists(), but figured out that adding the test for NULL
could not do any harm.


-- 
Stig Bjørlykke
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe