Re: [PATCH 0/3] libusbg: Fix potential issues found with coverity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On 13/05/14 09:58, Krzysztof Opasiak wrote:
Please excuse me but I have missed this question yesterday.

No problem, I was helping a friend move so did not have time to answer your other questions.;)


If you are talking about b returned from usbg_get_binding() don't be
afraid because it is only pointer to value stored in c->bindings list.
Then, in usbg_add_config_function() new b is allocated and we try to
create suitable symbolic link. If it fails b is freed in this function
otherwise it is added to c->bindings list using
INSERT_TAILQ_STRING_ORDER() macro. The whole list is freed in
usbg_free_config() while removing whole configuration and in suitable
places like removing function from configuration - usbg_rm_binding() (in
pending patches).

Yep that is exactly what I meant. So I read the code right. It was just not immediately clear to me, so I thought it would be good to double-check.

Cheers,

Philippe

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux