Stefan Sperling has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/12175 )

Change subject: log IPA tags correctly
......................................................................

log IPA tags correctly

Due to apparent copy-paste errors, wrong values were being
logged for IPA tags in ipa_bts_id_resp().

Fixes: 4c57eef663e870247cb7a72130a50750317a44b9
Change-Id: I29a0401db0760219b9b9176709a88d589312261b
Related: OS#3355
---
M src/input/ipaccess.c
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Max: Looks good to me, but someone else must approve
  Pau Espin Pedrol: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/input/ipaccess.c b/src/input/ipaccess.c
index 8c02996..136aa79 100644
--- a/src/input/ipaccess.c
+++ b/src/input/ipaccess.c
@@ -603,15 +603,15 @@
                        break;
                case IPAC_IDTAG_LOCATION1:
                        if (dev->location1)
-                               osmo_strlcpy(str, dev->swversion, sizeof(str));
+                               osmo_strlcpy(str, dev->location1, sizeof(str));
                        break;
                case IPAC_IDTAG_LOCATION2:
                        if (dev->location2)
-                               osmo_strlcpy(str, dev->swversion, sizeof(str));
+                               osmo_strlcpy(str, dev->location2, sizeof(str));
                        break;
                case IPAC_IDTAG_EQUIPVERS:
                        if (dev->equipvers)
-                               osmo_strlcpy(str, dev->swversion, sizeof(str));
+                               osmo_strlcpy(str, dev->equipvers, sizeof(str));
                        break;
                case IPAC_IDTAG_SWVERSION:
                        if (dev->swversion)
@@ -627,7 +627,7 @@
                        break;
                case IPAC_IDTAG_SERNR:
                        if (dev->serno)
-                               osmo_strlcpy(str, dev->swversion, sizeof(str));
+                               osmo_strlcpy(str, dev->serno, sizeof(str));
                        break;
                default:
                        LOGP(DLINP, LOGL_NOTICE,

--
To view, visit https://gerrit.osmocom.org/12175
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I29a0401db0760219b9b9176709a88d589312261b
Gerrit-Change-Number: 12175
Gerrit-PatchSet: 1
Gerrit-Owner: Stefan Sperling <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Stefan Sperling <[email protected]>

Reply via email to