[ovs-dev] [PATCH 2/4] fatal-signal: Clean up code by using shash.
Ben Pfaff
blp at nicira.com
Mon Sep 21 23:41:45 UTC 2009
Justin Pettit <jpettit at nicira.com> writes:
> On Sep 21, 2009, at 2:53 PM, Ben Pfaff wrote:
>
>> This simplifies the code here and should speed it up, too, when
>> there are
>> lots of files to unlink on a fatal signal.
>
> I don't suppose you care, but "files" is never destroyed. It wasn't
> done when it was an array, either so it is consistent. It only really
> matters when do_unlink_files() is called and the program is probably
> on its way out anyways...
That is intentional and necessary. do_unlink_files() gets called
from an asynchronous signal handler. If the signal arrives
during a call to malloc() or free(), etc., then freeing the data
structure could cause a segmentation fault.
More information about the dev
mailing list