Code review comment for lp:~xnox/upstart/async-remove-duplicate-event

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

ISSUE #1:
 1733 //FIXME must run without TEST_ALLOC_SAFE
 1734 TEST_ALLOC_SAFE {
 1735 event_poll ();
 1736 }

Deeply puzzled by this one:

Memory allocation fails during:
job->process_data = nih_alloc (...)
we then free the job:
nih_free (job);
jump to job_destory:
nih_list_destroy (&job->entry)

And get a segfault here....

« Back to merge proposal