discomfitor pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=ded3b767231b87bac0f135b7d923832b46db97bb

commit ded3b767231b87bac0f135b7d923832b46db97bb
Author: Mike Blumenkrantz <zm...@samsung.com>
Date:   Thu Mar 13 11:24:30 2014 -0400

    make connman if statements more clear for freeing data
    
    CID 1039738
---
 src/modules/connman/e_connman.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/modules/connman/e_connman.c b/src/modules/connman/e_connman.c
index a17fcda..e6595a3 100644
--- a/src/modules/connman/e_connman.c
+++ b/src/modules/connman/e_connman.c
@@ -242,12 +242,12 @@ static void _service_free(struct Connman_Service *cs)
         eldbus_pending_cancel(cs->pending.connect);
         free(cs->pending.data);
      }
-   if (cs->pending.disconnect)
+   else if (cs->pending.disconnect)
      {
         eldbus_pending_cancel(cs->pending.disconnect);
         free(cs->pending.data);
      }
-   if (cs->pending.remov)
+   else if (cs->pending.remov)
      {
         eldbus_pending_cancel(cs->pending.remov);
         free(cs->pending.data);

-- 


Reply via email to