discover/event: Ensure event struct exists for async callers
authorSamuel Mendoza-Jonas <sam@mendozajonas.com>
Thu, 1 Sep 2016 04:32:03 +0000 (14:32 +1000)
committerSamuel Mendoza-Jonas <sam@mendozajonas.com>
Thu, 1 Sep 2016 04:59:05 +0000 (14:59 +1000)
commita50d5fe279db71cf85fabeb675c99b167ec63dcb
tree2f7058badc87b030053ecb63c7312c3894ccf02c
parentccb478ac2e5b1e24ebb6af4130fdd37e1b36babb
discover/event: Ensure event struct exists for async callers

When handling an event, user_event_handle_message() creates an event
struct with relevant parameters. Once user_event_handle_message() is
finished it frees the struct.
However in the case of a dhcp or add_url event, asynchronous jobs may be
spawned that will later reference the event struct. In particular this
becomes a problem when pxe_process_pair() handles an IPAPPEND name/value
pair and tries to access event->device.

In the case of dhcp and add_url events, we avoid this by changing the
event struct's talloc parent to the discover_context struct which
persists until all async pxe jobs have completed.

Signed-off-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>
discover/device-handler.c
discover/user-event.c