[MERGED] openggsn[master]: ippool: Extend pool to work with /64 prefixes

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: ippool: Extend pool to work with /64 prefixes .. ippool: Extend pool to work with /64 prefixes In IPv6 GPRS, we actually don't want to allocate an

openggsn[master]: ippool: Extend pool to work with /64 prefixes

2017-08-09 Thread Harald Welte
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3457 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0cf700b6baf195a2e5fbea000531f801acaaa443 Gerrit-PatchSet: 2 Gerrit-Project: openggsn Gerrit-Branch: master

libosmo-sccp[master]: safer osmo_sccp_make_addr_pc_ssn(): clear all values

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3341 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I37395169d364050e45e99793c495a9848a4f8601 Gerrit-PatchSet: 1 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

[MERGED] libosmo-sccp[master]: safer osmo_sccp_make_addr_pc_ssn(): clear all values

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: safer osmo_sccp_make_addr_pc_ssn(): clear all values .. safer osmo_sccp_make_addr_pc_ssn(): clear all values Make sure that all values of the resulting

[MERGED] libosmo-sccp[master]: vty: hide SG commands when running as ASP

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: vty: hide SG commands when running as ASP .. vty: hide SG commands when running as ASP The SG (signal gateway) requires an extra set of VTY commands in

[MERGED] libosmo-sccp[master]: sccp: fix creating of the default route in simple client

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: sccp: fix creating of the default route in simple client .. sccp: fix creating of the default route in simple client The simple client

libosmo-sccp[master]: sccp: fix creating of the default route in simple client

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3452 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7516fa5509bade4ba10e9d8eaa3e46cfe3696b3a Gerrit-PatchSet: 1 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

libosmo-sccp[master]: vty: hide SG commands when running as ASP

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3453 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I707bcd790f4c6d0f618e41163b51d2179ea3aeaf Gerrit-PatchSet: 1 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

libosmocore[master]: Add osmo_gprs_{ul, dl}_block_size_{bits, bytes} functions

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3454 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I05ff75ef7dfae639886bbd09fe35f03a8af9d988 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master

[MERGED] libosmocore[master]: tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs .. tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs In the Protocol Configuration

[MERGED] libosmocore[master]: Add osmo_gprs_{ul, dl}_block_size_{bits, bytes} functions

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: Add osmo_gprs_{ul,dl}_block_size_{bits,bytes} functions .. Add osmo_gprs_{ul,dl}_block_size_{bits,bytes} functions Those functions can be used to look

[PATCH] openggsn[master]: Support setting TUN device IPv6 address + prefix

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3408 to look at the new patch set (#7). Support setting TUN device IPv6 address + prefix As we can now have PDP contexts with IPv6 user IP payload, it is useful to extend the TUN related

[PATCH] openggsn[master]: IPv6: in46_addr: OSMO_ASSERT() in case of unsupported calls

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3460 to look at the new patch set (#2). IPv6: in46_addr: OSMO_ASSERT() in case of unsupported calls There's a bit of trickery with the ip_pool and it's "lengty=8" IPv6 prefix handling,

[PATCH] openggsn[master]: IPv6: Implement IPv6 prefix assignment via ICMPv6 router adv...

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3458 to look at the new patch set (#2). IPv6: Implement IPv6 prefix assignment via ICMPv6 router advertisement The 3GPP specs are quite strange when it comes to how an IPv6 address or

[PATCH] openggsn[master]: IPv6: Support PCO for IPv6 DNS addresses

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3459 to look at the new patch set (#2). IPv6: Support PCO for IPv6 DNS addresses In IPv6, DNS server information is not passed along as IPCP6 like in IPv5 with IPCP. The reason is that

libosmocore[master]: tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3455 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9799130e2eba8fae8c4480fbb8a900c30232b694 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master

[MERGED] openggsn[master]: ippool_new(): const-ify input arguments

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: ippool_new(): const-ify input arguments .. ippool_new(): const-ify input arguments Change-Id: If3e53584e8c9c1f06bba4c183c9fd65fae913904 --- M

[MERGED] openggsn[master]: lib/tun.h: Remove non-endian-safe redefinition of IP header

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: lib/tun.h: Remove non-endian-safe redefinition of IP header .. lib/tun.h: Remove non-endian-safe redefinition of IP header We can simply use 'struct

[MERGED] openggsn[master]: ippool: Add IPv6 support to IP pool implementation

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: ippool: Add IPv6 support to IP pool implementation .. ippool: Add IPv6 support to IP pool implementation Extend the IP pool implementation to be able to

[MERGED] openggsn[master]: ggsn: Send proper errors in create_context_ind()

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: ggsn: Send proper errors in create_context_ind() .. ggsn: Send proper errors in create_context_ind() When we receive PDP context requests for unknown

[MERGED] openggsn[master]: IPv6 support for user IP

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: IPv6 support for user IP .. IPv6 support for user IP This patch enables the use of IPv6 PDP contexts. The phone will have to request an IPv6

[MERGED] openggsn[master]: Proper fix for "Force Dynamic IP" in Create PDP CTX

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: Proper fix for "Force Dynamic IP" in Create PDP CTX .. Proper fix for "Force Dynamic IP" in Create PDP CTX An EUA length of *2* octets indicates dynamic

openggsn[master]: in46_addr: Add new function in46a_prefix_equal()

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3456 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic993d8abdc90897cb55276f01ae3b8a5eadf5a0d Gerrit-PatchSet: 1 Gerrit-Project: openggsn Gerrit-Branch: master

[MERGED] openggsn[master]: in46_addr: Add new function in46a_prefix_equal()

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: in46_addr: Add new function in46a_prefix_equal() .. in46_addr: Add new function in46a_prefix_equal() This function is used to compare an IPv6 address

openggsn[master]: IPv6 support for user IP

2017-08-09 Thread Harald Welte
Patch Set 6: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3406 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I22c3bf32a98e5daf99d6eaeac8c9f95cc7574774 Gerrit-PatchSet: 6 Gerrit-Project: openggsn Gerrit-Branch: master

[MERGED] osmo-iuh[master]: make point codes configurable by SCCP address book

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: make point codes configurable by SCCP address book .. make point codes configurable by SCCP address book In the vty config, use the SCCP address book to

openggsn[master]: ippool: Add IPv6 support to IP pool implementation

2017-08-09 Thread Harald Welte
Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3403 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib98cc4bf634d6be9a7bf8c03a24e629455fcafc8 Gerrit-PatchSet: 5 Gerrit-Project: openggsn Gerrit-Branch: master

osmo-iuh[master]: make point codes configurable by SCCP address book

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3394 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I33a7ba11eb7c2d9a5dc74d10fb0cf04bf664477b Gerrit-PatchSet: 3 Gerrit-Project: osmo-iuh Gerrit-Branch: master

Build failure of network:osmocom:nightly/libosmocore in xUbuntu_16.04/i586

2017-08-09 Thread OBS Notification
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmocore/xUbuntu_16.04/i586 Package network:osmocom:nightly/libosmocore failed to build in xUbuntu_16.04/i586 Check out the package for editing: osc checkout network:osmocom:nightly libosmocore Last lines of

[PATCH] openggsn[master]: IPv6: Support PCO for IPv6 DNS addresses

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3459 IPv6: Support PCO for IPv6 DNS addresses In IPv6, DNS server information is not passed along as IPCP6 like in IPv5 with IPCP. The reason is that IPCP6 (for PPP) doesn't support passing DNS server information. Rather, the relevant RFCs indicate DHCPv6

[PATCH] openggsn[master]: IPv6: in46_addr: OSMO_ASSERT() in case of unsupported calls

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3460 IPv6: in46_addr: OSMO_ASSERT() in case of unsupported calls There's a bit of trickery with the ip_pool and it's "lengty=8" IPv6 prefix handling, let's make sure we don't accidentially call any support functions with addresses of wrong length.

[PATCH] openggsn[master]: in46_addr: Add new function in46a_prefix_equal()

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3456 in46_addr: Add new function in46a_prefix_equal() This function is used to compare an IPv6 address against another, using the smaller of the two prefix lengths. Change-Id: Ic993d8abdc90897cb55276f01ae3b8a5eadf5a0d --- M lib/in46_addr.c M

[PATCH] openggsn[master]: IPv6: Implement IPv6 prefix assignment via ICMPv6 router adv...

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3458 IPv6: Implement IPv6 prefix assignment via ICMPv6 router advertisement The 3GPP specs are quite strange when it comes to how an IPv6 address or rather prefix is assigned to an IPv6 PDP context. The designated method for allocating the IPv6 address via

[PATCH] openggsn[master]: IPv6 support for user IP

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3406 to look at the new patch set (#6). IPv6 support for user IP This patch enables the use of IPv6 PDP contexts. The phone will have to request an IPv6 End-user-Address, and the GGSN will

[PATCH] openggsn[master]: ggsn: Send proper errors in create_context_ind()

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3407 to look at the new patch set (#6). ggsn: Send proper errors in create_context_ind() When we receive PDP context requests for unknown PDP types or if we run out of dynamic addresses, we

[PATCH] openggsn[master]: ippool_new(): const-ify input arguments

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3405 to look at the new patch set (#6). ippool_new(): const-ify input arguments Change-Id: If3e53584e8c9c1f06bba4c183c9fd65fae913904 --- M lib/ippool.c M lib/ippool.h 2 files changed, 2

[PATCH] openggsn[master]: ippool: Extend pool to work with /64 prefixes

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3457 ippool: Extend pool to work with /64 prefixes In IPv6 GPRS, we actually don't want to allocate an individual v6 address (like in IPv4), but we want to allocate a prefix. The standard prefix lengh is 8 bytes, i.e. a /64 prefix. This patch extends the

[PATCH] openggsn[master]: lib/tun.h: Remove non-endian-safe redefinition of IP header

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3404 to look at the new patch set (#6). lib/tun.h: Remove non-endian-safe redefinition of IP header We can simply use 'struct iphdr' from netinet/ip.h to achieve the same goal (and be

[PATCH] openggsn[master]: ippool: Add IPv6 support to IP pool implementation

2017-08-09 Thread Harald Welte
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3403 to look at the new patch set (#5). ippool: Add IPv6 support to IP pool implementation Extend the IP pool implementation to be able to manage both pools of 32bit addresses (IPv4) as

[PATCH] libosmocore[master]: tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3455 tlv: add [msgb_]t16lv_put() for 16bit tag + 8 bit len TLVs In the Protocol Configuration Options IE (see 3GPP TS 24.008 10.5.6.3) there is yet another new TLV format (derived from PPP IPCP/LCP/...) which uses 16bit tag and 8bit length. Let's add

[PATCH] libosmocore[master]: Add osmo_gprs_{ul, dl}_block_size_{bits, bytes} functions

2017-08-09 Thread Harald Welte
Review at https://gerrit.osmocom.org/3454 Add osmo_gprs_{ul,dl}_block_size_{bits,bytes} functions Those functions can be used to look up the size of (E)GPRS blocks. Change-Id: I05ff75ef7dfae639886bbd09fe35f03a8af9d988 --- M include/osmocom/gprs/gprs_rlc.h M src/gsm/gprs_rlc.c M

[PATCH] libosmo-sccp[master]: vty: hide SG commands when running as ASP

2017-08-09 Thread dexter
Review at https://gerrit.osmocom.org/3453 vty: hide SG commands when running as ASP The SG (signal gateway) requires an extra set of VTY commands in order to be fully configurable. These VTY commands do not make sense in a situation where libosmo-sccp is used to implement an ASP (application

[PATCH] libosmo-sccp[master]: sccp: fix creating of the default route in simple client

2017-08-09 Thread dexter
Review at https://gerrit.osmocom.org/3452 sccp: fix creating of the default route in simple client The simple client (osmo_sccp_simple_client_on_ss7_id() and osmo_sccp_simple_client) does not create a default route if the user has already created an AS via VTY. Check the presence of a default

[ABANDON] osmo-iuh[master]: apply ctx arg to osmo_ss7_vty_init_asp()

2017-08-09 Thread Neels Hofmeyr
Neels Hofmeyr has abandoned this change. Change subject: apply ctx arg to osmo_ss7_vty_init_asp() .. Abandoned now part of 'make point codes configurable by SCCP address book' https://gerrit.osmocom.org/3394 -- To view,

[PATCH] osmo-iuh[master]: make point codes configurable by SCCP address book

2017-08-09 Thread Neels Hofmeyr
Hello Harald Welte, Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3394 to look at the new patch set (#3). make point codes configurable by SCCP address book In the vty config, use the SCCP address book to configure the local and remote SCCP

[MERGED] openbsc[master]: libmsc: handle delivery ack via SMPP SUBMIT SM / send GSM 03...

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: handle delivery ack via SMPP SUBMIT SM / send GSM 03.40 status report .. libmsc: handle delivery ack via SMPP SUBMIT SM / send GSM 03.40

[MERGED] openbsc[master]: libmsc: support GSM 03.40 status report for nitb

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: support GSM 03.40 status report for nitb .. libmsc: support GSM 03.40 status report for nitb This patch adds support for GSM 03.40 in nitb

openbsc[master]: libmsc: handle delivery ack via SMPP SUBMIT SM / send GSM 03...

2017-08-09 Thread Pablo Neira Ayuso
Patch Set 4: Verified+1 -- To view, visit https://gerrit.osmocom.org/3437 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib70e534840308ed315f7add440351e649de3f907 Gerrit-PatchSet: 4 Gerrit-Project: openbsc Gerrit-Branch: master

[MERGED] openbsc[master]: utils: smpp_mirror: bounce Delivery Receipts as Delivery Ack...

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: utils: smpp_mirror: bounce Delivery Receipts as Delivery Acknowledgments .. utils: smpp_mirror: bounce Delivery Receipts as Delivery

openbsc[master]: utils: smpp_mirror: bounce Delivery Receipts as Delivery Ack...

2017-08-09 Thread Pablo Neira Ayuso
Patch Set 4: Verified+1 -- To view, visit https://gerrit.osmocom.org/3436 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I877e228d8e174430f700631edbf9955972da7892 Gerrit-PatchSet: 4 Gerrit-Project: openbsc Gerrit-Branch: master

openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Pablo Neira Ayuso
Patch Set 4: Verified+1 (1 comment) https://gerrit.osmocom.org/#/c/3435/1/openbsc/src/libmsc/db.c File openbsc/src/libmsc/db.c: Line 375: /* Just like v3, but there is a new message reference field for status reports, > "just like v5"? Right, just like v3 actually. -- To view, visit

[MERGED] openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: update database to accomodate SMS status-report fields .. libmsc: update database to accomodate SMS status-report fields SMPP DELIVER_SM

osmo-pcu[master]: Add pcu-socket vty config

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3443 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9b3e1171da467519750b201849ec892a1e318129 Gerrit-PatchSet: 3 Gerrit-Project: osmo-pcu Gerrit-Branch: master

[MERGED] osmo-pcu[master]: Add pcu-socket vty config

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: Add pcu-socket vty config .. Add pcu-socket vty config osmo-bts already supports configuring a different path for the bts<->pcu socket by using the

[MERGED] osmo-bts[master]: TRX: Remove global variables, move SETBSIC/SETTSC handling i...

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: TRX: Remove global variables, move SETBSIC/SETTSC handling into phy_link .. TRX: Remove global variables, move SETBSIC/SETTSC handling into phy_link

[MERGED] osmo-bts[master]: l1sap/osmo-bts-sysmo: Improve logging

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: l1sap/osmo-bts-sysmo: Improve logging .. l1sap/osmo-bts-sysmo: Improve logging Change-Id: I339db0e5f3fd5e44bac974f2447afc80388802af --- M

osmo-bts[master]: TRX: Remove global variables, move SETBSIC/SETTSC handling i...

2017-08-09 Thread Harald Welte
Patch Set 4: Code-Review+2 (1 comment) https://gerrit.osmocom.org/#/c/3044/4//COMMIT_MSG Commit Message: PS4, Line 10: transceiver is a property > What is a property? property: https://en.wikipedia.org/wiki/Property_(programming) and no, my original sentence is what I want. "[The question of]

osmo-bts[master]: l1sap/osmo-bts-sysmo: Improve logging

2017-08-09 Thread Harald Welte
Patch Set 2: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3038 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I339db0e5f3fd5e44bac974f2447afc80388802af Gerrit-PatchSet: 2 Gerrit-Project: osmo-bts Gerrit-Branch: master

[MERGED] libosmo-sccp[master]: cosmetic: ws in log

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: cosmetic: ws in log .. cosmetic: ws in log Change-Id: If5a56ec170d4e91ce070f6fa9c9cfe5bbc121186 --- M src/sccp2sua.c 1 file changed, 1 insertion(+), 1

[MERGED] libosmo-sccp[master]: sccp-addr vty: set RI to SSN_PC when setting a point-code

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: sccp-addr vty: set RI to SSN_PC when setting a point-code .. sccp-addr vty: set RI to SSN_PC when setting a point-code Change-Id:

openbsc[master]: libmsc: support GSM 03.40 status report for nitb

2017-08-09 Thread Keith Whyte
Patch Set 4: > Patch Set 4: Verified+1 Not sure why I am permitted to Verify this... :-/ -- To view, visit https://gerrit.osmocom.org/3438 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5cc7bb4ebadde0940f44d10c3df34707b0615160

[MERGED] libosmo-sccp[master]: add osmo_sccp_addr_name() and three value_string[]s

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: add osmo_sccp_addr_name() and three value_string[]s .. add osmo_sccp_addr_name() and three value_string[]s osmo_sccp_addr_dump() just prints the raw

[MERGED] libosmo-sccp[master]: constify ss7_instance arg of osmo_ss7_pointcode_print()

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: constify ss7_instance arg of osmo_ss7_pointcode_print() .. constify ss7_instance arg of osmo_ss7_pointcode_print() Change-Id:

[MERGED] libosmo-sccp[master]: fix osmo_sccp_addr_dump(): actually print RI

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: fix osmo_sccp_addr_dump(): actually print RI .. fix osmo_sccp_addr_dump(): actually print RI Adjust xua_test accordingly. Change-Id:

[MERGED] libosmo-sccp[master]: introduce OSMO_SCCP_RI_NONE to indicate unset RI

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: introduce OSMO_SCCP_RI_NONE to indicate unset RI .. introduce OSMO_SCCP_RI_NONE to indicate unset RI Allows to automatically set an RI in future change

[MERGED] libosmo-sccp[master]: sccp_helpers: check buf len in append_to_buf()

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: sccp_helpers: check buf len in append_to_buf() .. sccp_helpers: check buf len in append_to_buf() Change-Id: I2e6d656871f952be8e719573fedf2154832841d7

[MERGED] libosmo-sccp[master]: consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user .. consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user A previous patch added ss7_instance

libosmo-sccp[master]: ensure valid primary_pc in osmo_ss7_instance

2017-08-09 Thread Harald Welte
Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3355 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7f0f0c89b7335d9da24161bfac8234be214ca00c Gerrit-PatchSet: 4 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

libosmo-sccp[master]: consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3451 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8684c9b559712072c772012890bbf7efa7c8eb35 Gerrit-PatchSet: 1 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

libosmo-sccp[master]: add OSMO_SS7_PC_INVALID, add osmo_ss7_pc_is_valid()

2017-08-09 Thread Harald Welte
Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3450 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib5715bf03a4de7713a7a809dfd821c700255ba8c Gerrit-PatchSet: 1 Gerrit-Project: libosmo-sccp Gerrit-Branch: master

openbsc[master]: libmsc: support GSM 03.40 status report for nitb

2017-08-09 Thread Keith Whyte
Patch Set 4: Verified+1 -- To view, visit https://gerrit.osmocom.org/3438 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5cc7bb4ebadde0940f44d10c3df34707b0615160 Gerrit-PatchSet: 4 Gerrit-Project: openbsc Gerrit-Branch: master

[MERGED] osmo-bts[master]: TRX: Remove bogus extern global variable declarations

2017-08-09 Thread Harald Welte
Harald Welte has submitted this change and it was merged. Change subject: TRX: Remove bogus extern global variable declarations .. TRX: Remove bogus extern global variable declarations Those global variable declarations for

openbsc[master]: libmsc: support GSM 03.40 status report for nitb

2017-08-09 Thread Harald Welte
Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3438 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5cc7bb4ebadde0940f44d10c3df34707b0615160 Gerrit-PatchSet: 4 Gerrit-Project: openbsc Gerrit-Branch: master

[MERGED] openbsc[master]: utils: smpp_mirror: temporarily munch SMPP delivery receipts

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: utils: smpp_mirror: temporarily munch SMPP delivery receipts .. utils: smpp_mirror: temporarily munch SMPP delivery receipts Just munch and log

[MERGED] openbsc[master]: utils: smpp_mirror: set registered_delivery field in SMPP SU...

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: utils: smpp_mirror: set registered_delivery field in SMPP SUBMIT_SM .. utils: smpp_mirror: set registered_delivery field in SMPP SUBMIT_SM To test

[MERGED] openbsc[master]: libmsc: add support for SMPP delivery receipts

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: add support for SMPP delivery receipts .. libmsc: add support for SMPP delivery receipts If the mobile phone requests a status report via

[MERGED] openbsc[master]: libmsc: report status report request flag from SMPP SUBMIT_SM

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: report status report request flag from SMPP SUBMIT_SM .. libmsc: report status report request flag from SMPP SUBMIT_SM Restore the sms

[MERGED] openbsc[master]: libmsc: missing bit shift in status report flag when stored ...

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: missing bit shift in status report flag when stored in sms object .. libmsc: missing bit shift in status report flag when stored in sms

openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Harald Welte
Patch Set 4: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3435 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7276d356d805a83ebeec72b02c8563b7135ea0b6 Gerrit-PatchSet: 4 Gerrit-Project: openbsc Gerrit-Branch: master

[MERGED] openbsc[master]: libmsc: set registered_delivery field in SMPP 3.4 DELIVER_SM...

2017-08-09 Thread Pablo Neira Ayuso
Pablo Neira Ayuso has submitted this change and it was merged. Change subject: libmsc: set registered_delivery field in SMPP 3.4 DELIVER_SM messages .. libmsc: set registered_delivery field in SMPP 3.4 DELIVER_SM messages

openbsc[master]: libmsc: set registered_delivery field in SMPP 3.4 DELIVER_SM...

2017-08-09 Thread Harald Welte
Patch Set 2: Verified+1 -- To view, visit https://gerrit.osmocom.org/3428 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I59af60fa89cd10ae973c5e122789e3e03e3728ee Gerrit-PatchSet: 2 Gerrit-Project: openbsc Gerrit-Branch: master

[PATCH] openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Pablo Neira Ayuso
Hello Neels Hofmeyr, Harald Welte, Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3435 to look at the new patch set (#4). libmsc: update database to accomodate SMS status-report fields SMPP DELIVER_SM messages with esm_class = Delivery

libasn1c[master]: add /debian to build a package

2017-08-09 Thread Neels Hofmeyr
Patch Set 1: Code-Review-1 lynxis, let's enhance the copyright notice as in above comments -- To view, visit https://gerrit.osmocom.org/3440 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I135d975b84027d51cb1dc534e04dd314bbd20289

libosmo-sccp[master]: ensure valid primary_pc in osmo_ss7_instance

2017-08-09 Thread Neels Hofmeyr
Patch Set 4: Now using 0x to mark invalid PC, this commit spread out into three: adding the invalid value, ensuring set primary_pc, and then adjusting osmo_sccp_user to use the same invalidity marking. -- To view, visit https://gerrit.osmocom.org/3355 To unsubscribe, visit

[PATCH] libosmo-sccp[master]: add OSMO_SS7_PC_INVALID, add osmo_ss7_pc_is_valid()

2017-08-09 Thread Neels Hofmeyr
Review at https://gerrit.osmocom.org/3450 add OSMO_SS7_PC_INVALID, add osmo_ss7_pc_is_valid() Introduce OSMO_SS7_PC_INVALID to mark an unset point code. Add static osmo_ss7_pc_is_valid() (name matches schema of osmo_ss7_pc_is_local()). In osmo_ss7_pointcode_print(), return "(no PC)" if

[PATCH] libosmo-sccp[master]: consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user

2017-08-09 Thread Neels Hofmeyr
Review at https://gerrit.osmocom.org/3451 consistency: use OSMO_SS7_PC_INVALID for osmo_sccp_user A previous patch added ss7_instance primary_pc validity checks by means of OSMO_SS7_PC_INVALID. To be consistent, also adjust sccp_user accordingly. (see I7f0f0c89b7335d9da24161bfac8234be214ca00c)

[PATCH] libosmo-sccp[master]: ensure valid primary_pc in osmo_ss7_instance

2017-08-09 Thread Neels Hofmeyr
Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3355 to look at the new patch set (#4). ensure valid primary_pc in osmo_ss7_instance Initialize osmo_ss7_instance.cfg.primary_pc = OSMO_SS7_PC_INVALID. Adjust all code paths using

openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Keith Whyte
Patch Set 3: > Can it work here? I don't believe so, SQLITE3 ALTER TABLE is limited to adding new columns to the end of the table. Probably why it's done like this in previous db version upgrades. https://www.sqlite.org/lang_altertable.html -- To view, visit https://gerrit.osmocom.org/3435

osmo-bts[master]: TRX: Remove bogus extern global variable declarations

2017-08-09 Thread Vadim Yanitskiy
Patch Set 5: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/3045 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I94b5f934fc3bd00b0467d90029d3053b16594186 Gerrit-PatchSet: 5 Gerrit-Project: osmo-bts Gerrit-Branch: master

openbsc[master]: libcommon: Fix log output for bts>0.

2017-08-09 Thread Alexander Chemeris
Patch Set 2: Hi Harald, thank you for a reminder. I'll try to find some time to extend the patch. -- To view, visit https://gerrit.osmocom.org/3185 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id:

[PATCH] osmo-bts[master]: TRX: Remove bogus extern global variable declarations

2017-08-09 Thread Harald Welte
Hello Vadim Yanitskiy, Max, Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/3045 to look at the new patch set (#5). TRX: Remove bogus extern global variable declarations Those global variable declarations for non-existing variables were

osmo-bts[master]: TRX: Remove bogus extern global variable declarations

2017-08-09 Thread Harald Welte
Patch Set 5: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3045 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I94b5f934fc3bd00b0467d90029d3053b16594186 Gerrit-PatchSet: 5 Gerrit-Project: osmo-bts Gerrit-Branch: master

openbsc[master]: SGSN: Fix RAN information exposed on GTP during PDP CTX CREATE

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3400 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6c79e42c5e08b28fe8182555302a5505fbbaa313 Gerrit-PatchSet: 3 Gerrit-Project: openbsc Gerrit-Branch: master

osmo-bts[master]: TRX: Remove bogus extern global variable declarations

2017-08-09 Thread Vadim Yanitskiy
Patch Set 4: Code-Review+1 Maxim has added the 'Related: OS#1848' label in the previous patch set. I think it makes sense to keep it. -- To view, visit https://gerrit.osmocom.org/3045 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id:

openbsc[master]: libmsc: report status report request flag from SMPP SUBMIT_SM

2017-08-09 Thread Pablo Neira Ayuso
Patch Set 3: Verified+1 -- To view, visit https://gerrit.osmocom.org/3429 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iac05252253f8933a3875b4904599b7a225191a4b Gerrit-PatchSet: 3 Gerrit-Project: openbsc Gerrit-Branch: master

openbsc[master]: libmsc: handle delivery ack via SMPP SUBMIT SM / send GSM 03...

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3437 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib70e534840308ed315f7add440351e649de3f907 Gerrit-PatchSet: 3 Gerrit-Project: openbsc Gerrit-Branch: master

openbsc[master]: utils: smpp_mirror: bounce Delivery Receipts as Delivery Ack...

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3436 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I877e228d8e174430f700631edbf9955972da7892 Gerrit-PatchSet: 3 Gerrit-Project: openbsc Gerrit-Branch: master

openbsc[master]: libmsc: add support for SMPP delivery receipts

2017-08-09 Thread Harald Welte
Patch Set 2: Code-Review+2 (1 comment) https://gerrit.osmocom.org/#/c/3434/1/openbsc/src/libmsc/gsm_04_11.c File openbsc/src/libmsc/gsm_04_11.c: Line 637: rc = sms_route_mt_sms(conn, sms_report); > if mem is up, the program will likely not do *anything* useful anymore. Ins It is our

openbsc[master]: libmsc: report status report request flag from SMPP SUBMIT_SM

2017-08-09 Thread Harald Welte
Patch Set 3: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/3429 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iac05252253f8933a3875b4904599b7a225191a4b Gerrit-PatchSet: 3 Gerrit-Project: openbsc Gerrit-Branch: master

openbsc[master]: libmsc: update database to accomodate SMS status-report fields

2017-08-09 Thread Holger Freyther
Patch Set 3: Have you considered using ALTER TABLE? Can it work here? -- To view, visit https://gerrit.osmocom.org/3435 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7276d356d805a83ebeec72b02c8563b7135ea0b6 Gerrit-PatchSet: 3

  1   2   >