[systemd-devel] [PATCH 1/2] endpoint: free the policy_db on error path

2014-03-23 Thread Djalal Harouni
Signed-off-by: Djalal Harouni tix...@opendz.org
---
 endpoint.c | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/endpoint.c b/endpoint.c
index 465ae31..5afabf9 100644
--- a/endpoint.c
+++ b/endpoint.c
@@ -236,7 +236,7 @@ int kdbus_ep_new(struct kdbus_bus *bus, const char *name,
if (bus-disconnected) {
mutex_unlock(bus-lock);
ret = -ESHUTDOWN;
-   goto exit_dev_unregister;
+   goto exit_policy_db_free;
}
e-id = ++bus-ep_seq_last;
e-bus = kdbus_bus_ref(bus);
@@ -247,6 +247,9 @@ int kdbus_ep_new(struct kdbus_bus *bus, const char *name,
*ep = e;
return 0;
 
+exit_policy_db_free:
+   if (policy)
+   kdbus_policy_db_free(e-policy_db);
 exit_dev_unregister:
device_unregister(e-dev);
 exit_idr:
-- 
1.8.5.3

___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel


Re: [systemd-devel] [PATCH 1/2] endpoint: free the policy_db on error path

2014-03-23 Thread Kay Sievers
On Sun, Mar 23, 2014 at 11:06 PM, Djalal Harouni tix...@opendz.org wrote:
 Signed-off-by: Djalal Harouni tix...@opendz.org
 ---
  endpoint.c | 5 -
  1 file changed, 4 insertions(+), 1 deletion(-)

Applied.

Thanks,
Kay
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel