tree 6e195847d8e4988e381c98f7fad3aae82ba9d035
parent bca0ef6cd9ba916c67f4bc06a1a7c49ff41ba935
author Philipp Maier <pmaier@sysmocom.de> 1532016477 +0200
committer Philipp Maier <pmaier@sysmocom.de> 1532016923 +0200

protocol: prevent unnecessary null pointer deref

The function setup_rtp_processing() in mgcp_protocol.c executes a
function pointer setup_rtp_processing_cb(). The function pointer
gets two struct mgcp_rtp_end pointers as parameter. To get those
parameters it has to dereference them from struct mgcp_conn_rtp
pointers. The variable conn_src is such a struct pointer and there
are conditions where this pointer may be NULL. The function at the
function pointer should get the conn pointers directly instead of
the dereferenced end (rtp) pointers. This also gives additional
flexibility to the implementation behind the function pointer,
which is not yet defined (the function pointer points always to
a stub function since we donot support transcoding yet.

- give conn pointers directly to setup_rtp_processing_cb() insed
  of dereferencing conn_src->end

Change-Id: Id46e9bfba88613387026639eb4957221cce6820a
Closes OS#3406