Subject: openbsc.git branch hfreyther/jerlbeck/mgcp-cleanups created. 0.13.0-197-gadf2c2c
From: [email-masked]
To: [email-masked]
Date: Wed, 4 Dec 2013 13:19: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, hfreyther/jerlbeck/mgcp-cleanups has been created
        at  adf2c2c2e7fd29c1e88a9e1142c4b50d78892d99 (commit)

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

commit adf2c2c2e7fd29c1e88a9e1142c4b50d78892d99
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=de61579174b43b4aeaaa1e1473da60a94e9a76b1

commit de61579174b43b4aeaaa1e1473da60a94e9a76b1
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=78a5b0b8565856c5268989d5ac29e1f17106f3ab

commit 78a5b0b8565856c5268989d5ac29e1f17106f3ab
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=d209cec23f274ec425880c3082ee9ebe8c653da5

commit d209cec23f274ec425880c3082ee9ebe8c653da5
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=45ae2c227ec43697e589876a9c403d7863d641f7

commit 45ae2c227ec43697e589876a9c403d7863d641f7
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=62d840f72c40be9cb295a5547e63cac74e2841fd

commit 62d840f72c40be9cb295a5547e63cac74e2841fd
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=73a40e4763bbb68677029119be53c48a27d4b159

commit 73a40e4763bbb68677029119be53c48a27d4b159
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

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

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