Attention is currently required from: osmith, laforge.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ci/+/28349 )
Change subject: update git URLs (git -> https; gitea/gerrit)
......................................................................
Patch Set 3:
(3 comments)
File ansible/roles/osmocom-workstation/tasks/git-repos.yml:
https://gerrit.osmocom.org/c/osmo-ci/+/28349/comment/1f3e682e_7e62646e
PS3, Line 7: repo: https://git.osmocom.org/{{ item }}.git
> Need to remove ".git" at the end: […]
Done in the next patchset.
File jobs/osmo-gsm-tester-runner.yml:
https://gerrit.osmocom.org/c/osmo-ci/+/28349/comment/d98969b4_df37791c
PS3, Line 35: url: https://gitea.osmocom.org/cellular-infrastructure/osmo-gsm-tester
> it seems these are randomly used? […]
I also find this inconsistency weird and confusing.
Why not simply use https://git.osmocom.org everywhere?
File scripts/osmo-depcheck/dependencies.py:
https://gerrit.osmocom.org/c/osmo-ci/+/28349/comment/9cfd340b_51ed998e
PS3, Line 17: httts
> https
Done in the next patchset.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/28349
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ia44e4127ff87a3d4d747225d3258188a0a891f1b
Gerrit-Change-Number: 28349
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Fri, 12 Aug 2022 20:36:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Gerrit-MessageType: comment
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/29071 )
Change subject: debian: clarify license/copyright for CSN.1 codec by ST-Ericsson
......................................................................
debian: clarify license/copyright for CSN.1 codec by ST-Ericsson
Change-Id: Ia76edccdfc14b06297fe7a306bc19ca52c7db3bd
---
M debian/copyright
1 file changed, 12 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmo-gprs refs/changes/71/29071/1
diff --git a/debian/copyright b/debian/copyright
index 670cede..718bd5a 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -5,6 +5,18 @@
Files: *
Copyright: 2022 sysmocom - s.f.m.c. GmbH <info(a)sysmocom.de>
License: GPL-2.0+
+
+Files: src/csn1/csn1.c
+ src/csn1/csn1_enc.c
+ src/csn1/csn1_dec.c
+ src/rlcmac/gprs_rlcmac.c
+ include/osmocom/csn1/csn1.h
+ include/osmocom/gprs/rlcmac/gprs_rlcmac.h
+Copyright: 2011 Vincent Helfre
+ 2011 ST-Ericsson (Jari Sassi)
+License: GPL-2.0+
+
+License: GPL-2.0+
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU Affero General Public License as published by
the Free Software Foundation; either version 3 of the License, or
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/29071
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: Ia76edccdfc14b06297fe7a306bc19ca52c7db3bd
Gerrit-Change-Number: 29071
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newchange
pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29069 )
Change subject: OSMUX_Emulation: Add special error case for port events
......................................................................
OSMUX_Emulation: Add special error case for port events
This way it's easier to quickly spot there was a problem with the socket
connection.
Change-Id: I962bf4837a9e359576c42a51a9919891186c7100
---
M library/OSMUX_Emulation.ttcn
1 file changed, 6 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/69/29069/1
diff --git a/library/OSMUX_Emulation.ttcn b/library/OSMUX_Emulation.ttcn
index 5d9a95e..6189266 100644
--- a/library/OSMUX_Emulation.ttcn
+++ b/library/OSMUX_Emulation.ttcn
@@ -422,7 +422,7 @@
var OsmuxRxHandle rx_hdl;
var OsmuxTxHandle tx_hdl;
var octetstring payload_truncated;
-
+ var PortEvent port_event;
timer T_transmit := int2float(g_cfg.tx_duration_ms)/1000.0;
var Osmux_RecvFrom rx_osmux;
var PDU_Osmux_AMR rx_amr;
@@ -569,6 +569,11 @@
T_transmit.start;
}
+ [] OSMUX.receive(PortEvent:?) -> value port_event {
+ Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail,
+ log2str("Received unexpected port event from Osmux:", port_event));
+ }
+
/* fail on any unexpected messages */
[] OSMUX.receive {
Misc_Helpers.f_shutdown(__BFILE__, __LINE__, fail,
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/29069
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I962bf4837a9e359576c42a51a9919891186c7100
Gerrit-Change-Number: 29069
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newchange
laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/29054 )
Change subject: fix warnings: 'uninitialized'
......................................................................
fix warnings: 'uninitialized'
Though these can never be used uninitialized, initialize to NULL to
avoid compiler warnings like:
pfcp_msg.c:188:66: warning: 'h_no_seid' may be used uninitialized
Change-Id: Icb338b200fe3186ccd7fd3f502c1723f60947190
---
M src/libosmo-gtlv/gtlv_dec_enc.c
M src/libosmo-pfcp/pfcp_msg.c
2 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, approved
dexter: Looks good to me, approved
laforge: Looks good to me, approved
diff --git a/src/libosmo-gtlv/gtlv_dec_enc.c b/src/libosmo-gtlv/gtlv_dec_enc.c
index 2e5509a..c3e45ea 100644
--- a/src/libosmo-gtlv/gtlv_dec_enc.c
+++ b/src/libosmo-gtlv/gtlv_dec_enc.c
@@ -71,7 +71,7 @@
{
void *obj = MEMB(decoded_struct, obj_ofs);
const struct osmo_gtlv_coding *iec;
- unsigned int *multi_count_p;
+ unsigned int *multi_count_p = NULL;
/* To check for presence of mandatory IEs, need to keep a flag stack of seen ie_coding entries. This array has
* to have at least the nr of entries that the ie_coding array has. Let's allow up to this many ie_coding
diff --git a/src/libosmo-pfcp/pfcp_msg.c b/src/libosmo-pfcp/pfcp_msg.c
index 5f55575..098ffa2 100644
--- a/src/libosmo-pfcp/pfcp_msg.c
+++ b/src/libosmo-pfcp/pfcp_msg.c
@@ -158,8 +158,8 @@
static int enc_pfcp_header(struct msgb *msg, const struct osmo_pfcp_msg *m)
{
const struct osmo_pfcp_header_parsed *parsed = &m->h;
- struct osmo_pfcp_header_seid *h_seid;
- struct osmo_pfcp_header_no_seid *h_no_seid;
+ struct osmo_pfcp_header_seid *h_seid = NULL;
+ struct osmo_pfcp_header_no_seid *h_no_seid = NULL;
struct osmo_pfcp_header_common *c;
int rc;
--
To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/29054
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: Icb338b200fe3186ccd7fd3f502c1723f60947190
Gerrit-Change-Number: 29054
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/29051 )
Change subject: debian/control: fix confusion in description
......................................................................
debian/control: fix confusion in description
Change-Id: I380b1dd626b3e6a35f17ae09a6758bef59f51c84
---
M debian/control
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/debian/control b/debian/control
index d1f8b43..ca7eaa6 100644
--- a/debian/control
+++ b/debian/control
@@ -46,7 +46,7 @@
Section: libdevel
Depends: ${misc:Depends},
libosmo-gtlv0 (= ${binary:Version})
-Description: Development headers for libosmo-gtlv
+Description: Development files for libosmo-gtlv
libosmo-gtlv implements decoding, encoding and logging of protocols using a
Tag-Length-Value and/or Tag-Length-Instance-Value structure. This package in
particular contains the files for compiling a program using this library.
@@ -80,7 +80,7 @@
Section: libdevel
Depends: ${misc:Depends},
libosmo-pfcp0 (= ${binary:Version})
-Description: Development headers for libosmo-pfcp
+Description: Development files for libosmo-pfcp
libosmo-pfcp implements decoding, encoding and endpoint implementation of the
Packet Forwarding Control Protocol (3GPP TS 29.244). This package in particular
contains the files for compiling a program using this library.
--
To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/29051
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: I380b1dd626b3e6a35f17ae09a6758bef59f51c84
Gerrit-Change-Number: 29051
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged