Subject: openbsc.git branch master updated. 0.13.0-198-g598b1c7
From: [email-masked]
To: [email-masked]
Date: Thu, 5 Dec 2013 09:32:46 GMT

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)".

The branch, master has been updated
       via  598b1c72310d4fad7500206cc47fe00b0fad9ec9 (commit)
       via  78a9501cfda39c8413c6698a063a08405521160c (commit)
       via  3dff27d38ded0e902c93dbc025f5664c3ac1ccdf (commit)
       via  9107e2da13ef83ad8a5ae40aabe2fe6bfb816dde (commit)
       via  1771171e056a167c559c7f479512647642f518f9 (commit)
       via  a52ac66e5227a31a4f8ecec6aa38b124cf6cb82b (commit)
       via  2bee7f96ff46f2f7e563d7aca5c4703214b6cd0e (commit)
       via  a01bd60851c2d2b613cda6148fb1ca5976887a93 (commit)
      from  19d8742820d1b67110d3028e71350c700ef025c3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cgit.osmocom.org/openbsc/commit/?id=598b1c72310d4fad7500206cc47fe00b0fad9ec9

commit 598b1c72310d4fad7500206cc47fe00b0fad9ec9
Merge: 19d8742 78a9501
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Thu Dec 5 10:32:06 2013 +0100

    Merge branch 'jerlbeck/mgcp-cleanups'

http://cgit.osmocom.org/openbsc/commit/?id=78a9501cfda39c8413c6698a063a08405521160c

commit 78a9501cfda39c8413c6698a063a08405521160c
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:49 2013 +0100

    mgcp: Handle SDP in CRCX received by the MGW
    
    So far the SDP part of the CRCX message has been ignored by the MGW.
    
    This patch adds SDP parsing for this case, eventually updating the
    net end's payload type and connection parameters.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=3dff27d38ded0e902c93dbc025f5664c3ac1ccdf

commit 3dff27d38ded0e902c93dbc025f5664c3ac1ccdf
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:48 2013 +0100

    mgcp/nat: Take payload type from SDP data
    
    So far the payload type used in RTP streams has been taken from the
    trunk configuration in NAT mode.
    
    This patch changes the implementation to use the payload type
    announced in the SDP part of MGCP messages and responses. SDP
    descriptions more than one m=audio line are not yet supported
    properly (always the last one is taken).
    
    Ticket: OW#466
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=9107e2da13ef83ad8a5ae40aabe2fe6bfb816dde

commit 9107e2da13ef83ad8a5ae40aabe2fe6bfb816dde
Author: Jacob Erlbeck <[email-masked]>
Date:   Tue Dec 3 17:14:44 2013 +0100

    mgcp: NUL-terminate MGCP message
    
    The MGCP message isn't always NUL-terminated when arriving at
    mgcp_handle_message(). This may lead to undefined results.
    
    This patch ensures that the message text is NUL-terminated by
    setting *msg->tail to '\0' in mgcp_handle_message().
    
    Addresses:
    <000b> mgcp_protocol.c:642 Unhandled option: 'r'/114 on 0x3
    <000b> mgcp_protocol.c:593 Unhandled SDP option: '='/61 on 0x3
    <000b> mgcp_protocol.c:871 Unhandled option: '.'/46 on 0x2
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=1771171e056a167c559c7f479512647642f518f9

commit 1771171e056a167c559c7f479512647642f518f9
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:46 2013 +0100

    mgcp: Refactor MGCP/SDP parsing
    
    This patch separates the SDP parsing from the (message specific) MGCP
    parsing.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=a52ac66e5227a31a4f8ecec6aa38b124cf6cb82b

commit a52ac66e5227a31a4f8ecec6aa38b124cf6cb82b
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:45 2013 +0100

    mgcp: Add tests for payload types in MGCP messages
    
    These tests mainly check whether the SDP parsing works properly by
    looking at the payload type detected.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=2bee7f96ff46f2f7e563d7aca5c4703214b6cd0e

commit 2bee7f96ff46f2f7e563d7aca5c4703214b6cd0e
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:44 2013 +0100

    mgcp: Add new for_each_line macro that also returns empty lines
    
    This patch add the for_each_line macro based on a strline_r()
    function (similar to strtok_r()), that is also part of this patch.
    This strline_r() function is tolerant with respect to line endings,
    it supports CR-only, CRLF, and LF-only and any combinations thereof
    (note that a CRLF is always detected as a single line break).
    
    Similar to for_each_non_empty_line (the former for_each_line) where
    the 'save' pointer needed to be initialised by a call to strtok_r(),
    the new for_each_line macro expects, that the 'save' pointer has been
    initialised by a call to strline_r(). Also note, that
    for_each_line/strline_r and for_each_non_empty_line/strtok_r may use
    the 'save' pointer differently, so calls to them can not be mixed.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=a01bd60851c2d2b613cda6148fb1ca5976887a93

commit a01bd60851c2d2b613cda6148fb1ca5976887a93
Author: Jacob Erlbeck <[email-masked]>
Date:   Fri Nov 29 13:43:43 2013 +0100

    mgcp: Rename for_each_line to for_each_non_empty_line
    
    The implementation of for_each_line is based on strtok() and skips
    any sequence of CR and LF. Thus empty lines are never detected. There
    exists code which tests for an empty line to detect the beginning of
    the SDP part which is dead code currently (the parser works
    nevertheless due to other reasons). So the semantics of this macro
    have been misunderstood at least once.
    
    This patch renames the macro to reflect the semantics more precisely.
    
    Sponsored-by: On-Waves ehf

-----------------------------------------------------------------------

Summary of changes:
 openbsc/include/openbsc/bsc_nat.h         |   3 +-
 openbsc/src/libmgcp/mgcp_protocol.c       | 151 ++++++++++++++++++++++++------
 openbsc/src/osmo-bsc_nat/bsc_mgcp_utils.c |  17 +++-
 openbsc/tests/bsc-nat/bsc_data.c          |   5 +
 openbsc/tests/bsc-nat/bsc_nat_test.c      |  16 +++-
 openbsc/tests/mgcp/mgcp_test.c            | 117 ++++++++++++++++++++++-
 openbsc/tests/mgcp/mgcp_test.ok           |  14 +++
 7 files changed, 281 insertions(+), 42 deletions(-)

hooks/post-receive
-- 
The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)