diff options
| author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-05-16 19:17:20 +0700 |
|---|---|---|
| committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-05-16 19:22:19 +0700 |
| commit | 7726eaa4777c93f8931a1aa30d64cf6ee20ac394 (patch) | |
| tree | 1fb427f1333487b40d4853ab0404e969a000e4b6 /src | |
| parent | 71fd083c16caf1bb073ca52023c607faab073f5f (diff) | |
oml: fix oml_mo_tx_sw_act_rep(): do not allocate FOM header twice
This redundant call to msgb_put() appends five '00'O octets to the
"Software Activated Report" message, so they look like an unknown
attribute(s). I accidentally noticed this in Wireshark.
Change-Id: I7377575135f4a154572891d1b5b39ff814b97f38
Diffstat (limited to 'src')
| -rw-r--r-- | src/common/oml.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/common/oml.c b/src/common/oml.c index 7a62faae..7e3b8348 100644 --- a/src/common/oml.c +++ b/src/common/oml.c @@ -472,7 +472,6 @@ int oml_mo_tx_sw_act_rep(const struct gsm_abis_mo *mo) if (!nmsg) return -ENOMEM; - msgb_put(nmsg, sizeof(struct abis_om_fom_hdr)); return oml_mo_send_msg(mo, nmsg, NM_MT_SW_ACTIVATED_REP); } |