Fixed null-pointer dereference in smp plugin.

diff --git a/src/libcharon/plugins/smp/smp.c b/src/libcharon/plugins/smp/smp.c
index 1873d9621b4bb6730769dfd49eb362a3f96528aa..2b830012d82abbf64fd3882c93e9e78161d15277 100644 (file)
--- a/src/libcharon/plugins/smp/smp.c
+++ b/src/libcharon/plugins/smp/smp.c
@@ -461,17 +461,21 @@ static void request_control_initiate(xmlTextReaderPtr reader,
 
                /* <log> */
                xmlTextWriterStartElement(writer, "log");
-               peer = charon->backends->get_peer_cfg_by_name(charon->backends, (char*)str);
+               peer = charon->backends->get_peer_cfg_by_name(charon->backends,
+                                                                                                         (char*)str);
                if (peer)
                {
                        enumerator = peer->create_child_cfg_enumerator(peer);
                        if (ike)
                        {
-                               if (!enumerator->enumerate(enumerator, &child))
+                               if (enumerator->enumerate(enumerator, &child))
+                               {
+                                       child->get_ref(child);
+                               }
+                               else
                                {
                                        child = NULL;
                                }
-                               child->get_ref(child);
                        }
                        else
                        {