Subject: openbsc.git branch master updated. 0.13.0-357-g444d7d8
From: [email-masked]
To: [email-masked]
Date: Fri, 4 Apr 2014 11:07:38 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  444d7d8e65447ca871db00828d2eda1490c2bbe7 (commit)
       via  686191a1c9be0f6dcfe92782332c4507b70e37e1 (commit)
       via  90e9a445c86d0929d4a58e35a8db13d745609ef0 (commit)
       via  5a1b329a9e7065903f800cd18584c2c72a6e9a1a (commit)
       via  65762e045587ba74b08a27f7a268966f5d3363ee (commit)
       via  744568b569343aad6be413bed7ac6a8820cf3bd2 (commit)
       via  cfee952e23244dc92a3896e66f2463240983f2c7 (commit)
       via  1768a5765d3ed8b1ca2aa514c8b0cbfb92d024a7 (commit)
      from  4766524d17a5bedfb6d90cce144ffca73931f5a2 (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=444d7d8e65447ca871db00828d2eda1490c2bbe7

commit 444d7d8e65447ca871db00828d2eda1490c2bbe7
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 13:01:28 2014 +0200

    ipa: Fix compiler warnings about aliasing
    
    Use memcpy to copy from the OML message into the stack and then
    convert the network byte order.
    
    network_listen.c: In function ���test_rep���:
    network_listen.c:145:2: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      test_rep_len = ntohs(*(uint16_t *) &foh->data[3]);
      ^
    network_listen.c:153:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
       ferr_list_len = ntohs(*(uint16_t *) &foh->data[7]);
       ^
    network_listen.c:164:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
       ferr_list_len = ntohs(*(uint16_t *) &foh->data[7]);
       ^
    network_listen.c:130:11: warning: variable ���test_rep_len��� set but not used [-Wunused-but-set-variable]
      uint16_t test_rep_len, ferr_list_len;

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

commit 686191a1c9be0f6dcfe92782332c4507b70e37e1
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:56:34 2014 +0200

    oml: Fix compiler warning about aliasing
    
    Make the fill_fom_hdr return the header and use it throughout.
    
      CC       abis_nm.o
    In file included from ../../include/openbsc/debug.h:8:0,
                     from abis_nm.c:38:
    abis_nm.c: In function ���abis_nm_opstart���:
    abis_nm.c:1763:34: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
                                      ^
    /home/ich/install/openbsc/include/osmocom/core/logging.h:23:74: note: in definition of macro ���DEBUGP���
     #define DEBUGP(ss, fmt, args...) logp(ss, __FILE__, __LINE__, 0, fmt, ## args)
                                                                              ^
    abis_nm.c:1763:2: note: in expansion of macro ���abis_nm_debugp_foh���
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
      ^
    abis_nm.c:1763:34: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
                                      ^
    /home/ich/install/openbsc/include/osmocom/core/logging.h:23:74: note: in definition of macro ���DEBUGP���
     #define DEBUGP(ss, fmt, args...) logp(ss, __FILE__, __LINE__, 0, fmt, ## args)
                                                                              ^
    abis_nm.c:1763:2: note: in expansion of macro ���abis_nm_debugp_foh���
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
      ^
    abis_nm.c:1763:34: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
                                      ^
    /home/ich/install/openbsc/include/osmocom/core/logging.h:23:74: note: in definition of macro ���DEBUGP���
     #define DEBUGP(ss, fmt, args...) logp(ss, __FILE__, __LINE__, 0, fmt, ## args)
                                                                              ^
    abis_nm.c:1763:2: note: in expansion of macro ���abis_nm_debugp_foh���
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
      ^
    abis_nm.c:1763:34: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
                                      ^
    /home/ich/install/openbsc/include/osmocom/core/logging.h:23:74: note: in definition of macro ���DEBUGP���
     #define DEBUGP(ss, fmt, args...) logp(ss, __FILE__, __LINE__, 0, fmt, ## args)
                                                                              ^
    abis_nm.c:1763:2: note: in expansion of macro ���abis_nm_debugp_foh���
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
      ^
    abis_nm.c:1763:34: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);
                                      ^
    /home/ich/install/openbsc/include/osmocom/core/logging.h:23:74: note: in definition of macro ���DEBUGP���
     #define DEBUGP(ss, fmt, args...) logp(ss, __FILE__, __LINE__, 0, fmt, ## args)
                                                                              ^
    abis_nm.c:1763:2: note: in expansion of macro ���abis_nm_debugp_foh���
      abis_nm_debugp_foh(DNM, (struct abis_om_fom_hdr *) oh->data);

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

commit 90e9a445c86d0929d4a58e35a8db13d745609ef0
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:51:28 2014 +0200

    gprs: Fix compiler warnings in sgsn_main.c
    
    sgsn_main.c: In function ���main���:
    sgsn_main.c:345:2: warning: implicit declaration of function ���gprs_sndcp_vty_init��� [-Wimplicit-function-declaration]
      gprs_sndcp_vty_init();
      ^
    sgsn_main.c:354:2: warning: implicit declaration of function ���sgsn_gtp_init��� [-Wimplicit-function-declaration]
      rc = sgsn_gtp_init(&sgsn_inst);
      ^

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

commit 5a1b329a9e7065903f800cd18584c2c72a6e9a1a
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:49:38 2014 +0200

    gprs: Fix compiler warnings ini gprs_sndcp_vty.c
    
       CC       gprs_sndcp_vty.o
    gprs_sndcp_vty.c: In function ���vty_dump_sne���:
    gprs_sndcp_vty.c:46:15: warning: unused variable ���i��� [-Wunused-variable]
      unsigned int i;

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

commit 65762e045587ba74b08a27f7a268966f5d3363ee
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:49:00 2014 +0200

    gprs: Fix compiler warnings in sgsn_libgtp.c
    
      CC       sgsn_libgtp.o
    sgsn_libgtp.c: In function ���create_pdp_conf���:
    sgsn_libgtp.c:262:6: warning: variable ���rc��� set but not used [-Wunused-but-set-variable]
      int rc;
          ^
    sgsn_libgtp.c: In function ���cb_data_ind���:
    sgsn_libgtp.c:432:6: warning: variable ���rc��� set but not used [-Wunused-but-set-variable]
      int rc;

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

commit 744568b569343aad6be413bed7ac6a8820cf3bd2
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:47:32 2014 +0200

    gprs: Fix compiler warnings in the gprs_llc.c code
    
      CC       gprs_llc.o
    gprs_llc.c: In function ���t200_expired���:
    gprs_llc.c:322:2: warning: enumeration value ���GPRS_LLES_UNASSIGNED��� not handled in switch [-Wswitch]
      switch (lle->state) {
      ^
    gprs_llc.c:322:2: warning: enumeration value ���GPRS_LLES_ASSIGNED_ADM��� not handled in switch [-Wswitch]
    gprs_llc.c:322:2: warning: enumeration value ���GPRS_LLES_REMOTE_EST��� not handled in switch [-Wswitch]
    gprs_llc.c:322:2: warning: enumeration value ���GPRS_LLES_ABM��� not handled in switch [-Wswitch]
    gprs_llc.c:322:2: warning: enumeration value ���GPRS_LLES_TIMER_REC��� not handled in switch [-Wswitch]
    gprs_llc.c: In function ���gprs_llc_hdr_rx���:
    gprs_llc.c:564:2: warning: enumeration value ���GPRS_LLC_NULL��� not handled in switch [-Wswitch]
      switch (gph->cmd) {
      ^
    gprs_llc.c:564:2: warning: enumeration value ���GPRS_LLC_RR��� not handled in switch [-Wswitch]
    gprs_llc.c:564:2: warning: enumeration value ���GPRS_LLC_ACK��� not handled in switch [-Wswitch]
    gprs_llc.c:564:2: warning: enumeration value ���GPRS_LLC_RNR��� not handled in switch [-Wswitch]
    gprs_llc.c:564:2: warning: enumeration value ���GPRS_LLC_SACK��� not handled in switch [-Wswitch]
    gprs_llc.c: In function ���gprs_llc_rcvmsg���:
    gprs_llc.c:791:23: warning: unused variable ���udh��� [-Wunused-variable]
      struct bssgp_ud_hdr *udh = (struct bssgp_ud_hdr *) msgb_bssgph(msg);
                           ^
    gprs_llc.c: At top level:
    gprs_llc.c:311:13: warning: ���t200_expired��� defined but not used [-Wunused-function]
     static void t200_expired(void *data)
                 ^
    gprs_llc.c:337:13: warning: ���t201_expired��� defined but not used [-Wunused-function]
     static void t201_expired(void *data)

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

commit cfee952e23244dc92a3896e66f2463240983f2c7
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:43:08 2014 +0200

    gprs: Fix compiler warnings in gprs_sndcp.c
    
      CC       gprs_sndcp.o
    gprs_sndcp.c: In function ���defrag_input���:
    gprs_sndcp.c:188:25: warning: variable ���scomph��� set but not used [-Wunused-but-set-variable]
      struct sndcp_comp_hdr *scomph = NULL;
                             ^
    gprs_sndcp.c: In function ���sndcp_llunitdata_ind���:
    gprs_sndcp.c:512:11: warning: variable ���npdu_num��� set but not used [-Wunused-but-set-variable]
      uint16_t npdu_num;
               ^
    gprs_sndcp.c: At top level:
    gprs_sndcp.c:565:12: warning: ���sndcp_ll_reset_ind��� defined but not used [-Wunused-function]
     static int sndcp_ll_reset_ind(struct gprs_sndcp_entity *se)
                ^
    gprs_sndcp.c:573:12: warning: ���sndcp_ll_status_ind��� defined but not used [-Wunused-function]
     static int sndcp_ll_status_ind()
                ^

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

commit 1768a5765d3ed8b1ca2aa514c8b0cbfb92d024a7
Author: Holger Hans Peter Freyther <[email-masked]>
Date:   Fri Apr 4 12:40:34 2014 +0200

    gprs: Reduce the number of compiler warnings in gprs_gmm.c
    
      CC       gprs_gmm.o
    gprs_gmm.c: In function ���gsm48_tx_gmm_att_ack���:
    gprs_gmm.c:350:11: warning: unused variable ���ptsig��� [-Wunused-variable]
      uint8_t *ptsig, *mid;
               ^
    gprs_gmm.c: In function ���gsm48_rx_gmm_auth_ciph_resp���:
    gprs_gmm.c:524:6: warning: variable ���rc��� set but not used [-Wunused-but-set-variable]
      int rc;
          ^
    gprs_gmm.c: In function ���gsm48_rx_gmm_att_req���:
    gprs_gmm.c:703:9: warning: implicit declaration of function ���sgsn_acl_lookup��� [-Wimplicit-function-declaration]
             !sgsn_acl_lookup(mi_string))) {
             ^
    gprs_gmm.c:632:40: warning: variable ���old_ra_info��� set but not used [-Wunused-but-set-variable]
      uint8_t *cur = gh->data, *msnc, *mi, *old_ra_info, *ms_ra_acc_cap;
                                            ^
    gprs_gmm.c: In function ���gsm48_rx_gmm_ra_upd_req���:
    gprs_gmm.c:915:6: warning: variable ���rc��� set but not used [-Wunused-but-set-variable]
      int rc;
          ^
    gprs_gmm.c:910:11: warning: variable ���ms_ra_acc_cap��� set but not used [-Wunused-but-set-variable]
      uint8_t *ms_ra_acc_cap;
               ^
    gprs_gmm.c: At top level:
    gprs_gmm.c:458:12: warning: ���gsm48_tx_gmm_auth_ciph_req��� defined but not used [-Wunused-function]
     static int gsm48_tx_gmm_auth_ciph_req(struct sgsn_mm_ctx *mm, uint8_t *rand,
                ^
    gprs_gmm.c:501:12: warning: ���gsm48_tx_gmm_auth_ciph_rej��� defined but not used [-Wunused-function]
     static int gsm48_tx_gmm_auth_ciph_rej(struct sgsn_mm_ctx *mm)
                ^
    gprs_gmm.c:1169:13: warning: ���msgb_put_pdp_addr_ipv4��� defined but not used [-Wunused-function]
     static void msgb_put_pdp_addr_ipv4(struct msgb *msg, uint32_t ipaddr)
                 ^
    gprs_gmm.c:1180:13: warning: ���msgb_put_pdp_addr_ppp��� defined but not used [-Wunused-function]
     static void msgb_put_pdp_addr_ppp(struct msgb *msg)

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

Summary of changes:
 openbsc/include/openbsc/gprs_sgsn.h   | 10 ++++++++++
 openbsc/src/gprs/gprs_gmm.c           | 23 ++++++++++++-----------
 openbsc/src/gprs/gprs_llc.c           | 10 +++++++++-
 openbsc/src/gprs/gprs_sndcp.c         |  6 ++----
 openbsc/src/gprs/gprs_sndcp_vty.c     |  2 --
 openbsc/src/gprs/sgsn_libgtp.c        |  6 ++----
 openbsc/src/ipaccess/network_listen.c |  9 ++++++---
 openbsc/src/libbsc/abis_nm.c          |  8 +++++---
 8 files changed, 46 insertions(+), 28 deletions(-)

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