Hi,
On Thu, 2015-06-18 at 19:16 +0530, Maneesh Jain wrote:
> The patch is used to check if memory is allocated or not properly
> in following functions. In case memory is not allocated, return
> it from the function to avoid any potential crash.
>
> a. signal_peer_found
> b. signal_peer_changed
The dbus_malloc0 should be replaced with something more sensible, as
should g_try_new0 few lines above. If we run out of this small memory
allocations here, it's quite sure that we'll crash in some other part of
the code when doing the next g_new0. So why not just change this to
g_new0, which does not need any checking for the returned value?
Cheers,
Patrik
_______________________________________________
connman mailing list
[email protected]
https://lists.connman.net/mailman/listinfo/connman
