[PATCH] libbsc: fix message leaks on several error paths

[pablo]

From: Pablo Neira Ayuso <pa...@gnumonks.org>

This patch fixes several leak of msgbs in uncommon error paths.
---
 openbsc/src/libbsc/abis_nm.c |   13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/openbsc/src/libbsc/abis_nm.c b/openbsc/src/libbsc/abis_nm.c
index ee0dd60..2c8b3ee 100644
--- a/openbsc/src/libbsc/abis_nm.c
+++ b/openbsc/src/libbsc/abis_nm.c
@@ -635,13 +635,16 @@ int abis_nm_rcvmsg(struct msgb *msg)
        if (oh->placement != ABIS_OM_PLACEMENT_ONLY) {
                LOGP(DNM, LOGL_ERROR, "ABIS OML placement 0x%x not supported\n",
                        oh->placement);
-               if (oh->placement != ABIS_OM_PLACEMENT_FIRST)
-                       return -EINVAL;
+               if (oh->placement != ABIS_OM_PLACEMENT_FIRST) {
+                       rc = -EINVAL;
+                       goto err;
+               }
        }
        if (oh->sequence != 0) {
                LOGP(DNM, LOGL_ERROR, "ABIS OML sequence 0x%x != 0x00\n",
                        oh->sequence);
-               return -EINVAL;
+               rc = -EINVAL;
+               goto err;
        }
 #if 0
        unsigned int l2_len = msg->tail - (uint8_t *)msgb_l2(msg);
@@ -671,9 +674,9 @@ int abis_nm_rcvmsg(struct msgb *msg)
        default:
                LOGP(DNM, LOGL_ERROR, "unknown ABIS OML message discriminator 
0x%x\n",
                        oh->mdisc);
-               return -EINVAL;
+               rc = -EINVAL;
        }
-
+err:
        msgb_free(msg);
        return rc;
 }
-- 
1.7.10.4