Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 39318: /trunk/epan/ /trunk/epan/: fi

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
Date: Sun, 9 Oct 2011 20:53:11 +0200
On Sun, Oct 9, 2011 at 6:28 PM, Guy Harris <guy@xxxxxxxxxxxx> 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