[sr-dev] git:master:09f79753: modules/ims_dialog: check pointer before dereferencing

[jaybeepee]

Module: kamailio
Branch: master
Commit: 09f797539c1f57ced771a882af8a51d2aca14554
URL: https://github.com/kamailio/kamailio/commit/09f797539c1f57ced771a882af8a51d2aca14554

Author: jaybeepee <jason.penton at gmail.com>
Committer: jaybeepee <jason.penton at gmail.com>
Date: 2016-09-13T13:10:59+02:00

modules/ims_dialog: check pointer before dereferencing

---

Modified: modules/ims_dialog/dlg_handlers.c

---

Diff:  https://github.com/kamailio/kamailio/commit/09f797539c1f57ced771a882af8a51d2aca14554.diff
Patch: https://github.com/kamailio/kamailio/commit/09f797539c1f57ced771a882af8a51d2aca14554.patch

---

diff --git a/modules/ims_dialog/dlg_handlers.c b/modules/ims_dialog/dlg_handlers.c
index faf7678..394c779 100644
--- a/modules/ims_dialog/dlg_handlers.c
+++ b/modules/ims_dialog/dlg_handlers.c
@@ -1342,7 +1342,7 @@ void dlg_onreply(struct cell* t, int type, struct tmcb_params *param) {
 
         if (!dlg_out) {
             if (rpl->first_line.u.reply.statuscode < 299) { /*we don't care about failure responses to dialog - not necessary to create dialog out...*/
-                if (rpl->via1->branch && (rpl->via1->branch->value.len > 0)) {
+                if (rpl->via1->branch && (&rpl->via1->branch->value) && (rpl->via1->branch->value.len > 0)) {
                     branch = rpl->via1->branch->value;
                 }