On 17/10/2017 07:45, David Gibson wrote: >> >> post_realize_fail: >> + g_free(dev->canonical_path); > Should dev->canonical_path be set to NULL on this failure path? >
It should be fine because dev->pending_deleted_event can only become true if dev->realized has ever been set to true. But it's certainly cleaner to clear it, so I'll do the change. Queued the series, thanks. Paolo
signature.asc
Description: OpenPGP digital signature