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

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to