From notifications@github.com Thu Sep 19 09:28:30 2019
From: vipulpatel310 <notifications@github.com>
To: sr-dev@lists.kamailio.org
Subject: [sr-dev] [kamailio/kamailio] erlang: getting null value in erl_xbuff
 in response of erl_rpc request (#2069)
Date: Thu, 19 Sep 2019 00:28:22 -0700
Message-ID: <kamailio/kamailio/issues/2069@github.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1863389327=="

--===============1863389327==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

### Description
**I did:**
I have installed kamailio 5.2.4 in ubuntu system and configured kamailio scri=
pt to connect with erlang node also it sends rpc request to erlang node and g=
et response which I have printed in a logs.
...
 $erl_list(L) =3D "two";
 $erl_list(L) =3D "one";
 xlog("L_ALERT","Before RPC: $erl_list(args=3D>format)\n");
 erl_rpc("erlang", "list_to_tuple", "$erl_list(args)", "$erl_xbuff(repl)");
 xlogl("L_ALERT","type(reply): $erl_xbuff(repl=3D>type), format(repl): $erl_x=
buff(repl=3D>format)\n");
...

**Expected to happen:**
Response should be get from erlang node against RPC request and it should be =
store in erl_xbuff. And it should be print logs like: ALERT: <script>: 501:ty=
pe(reply): tuple, format(repl): {"one", "two"}

**Actually happened:**
Response is getting properly. but the problem is to store data into xbuff so =
it print logs like: ALERT: <script>: 501:type(reply): <null>, format(repl): <=
null>

### Troubleshooting
I have check with older version 4.3.0 and it works fine. getting this problem=
 after 5.2.0 version
So, I have compare 5.2.0 and 4.3.0 version code. I found one change in pv_xbu=
ff.c file which is creating a problem when I revert the change then it works =
fine.

#### Reproduction
It will be reproducible everytime. Just sends rpc request and get the value i=
n xbuff. check xbuff value

#### Log Messages
Problem logs:
```
ALERT: <script>: Before RPC: [["one", "two"]]
WARNING: <core> [core/mem/q_malloc.c:480]: qm_free(): WARNING: free(0) called=
 from erlang: pv_xbuff.c: xavp_decode(1209)
ALERT: <script>: 501:type(reply): <null>, format(repl): <null>
```
logs after change:
```
ALERT: <script>: Before RPC: [["one", "two"]]
WARNING: <core> [core/mem/q_malloc.c:480]: qm_free(): WARNING: free(0) called=
 from erlang: pv_xbuff.c: xavp_decode(1209)
ALERT: <script>: 501:type(reply): tuple, format(repl): {"one", "two"}
```

### Possible Solutions
revert the 6542a367638ec5f8a2b62994f8c309e7a2533962 commit. It will work fine

### Additional Information

```
version: kamailio 5.2.4 (x86_64/linux) 759867
flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISA=
BLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F=
_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_=
DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES
ADAPTIVE_WAIT_LOOPS=3D1024, MAX_RECV_BUFFER_SIZE 262144 MAX_URI_SIZE 1024, BU=
F_SIZE 65535, DEFAULT PKG_SIZE 8MB
poll method support: poll, epoll_lt, epoll_et, sigio_rt, select.
id: 759867=20
compiled on 11:52:55 Sep  4 2019 with gcc 7.4.0

```

* **Operating System**:
```
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04 LTS
Release:	18.04
Codename:	bionic

Linux hostname.domain.com 4.15.0-20-generic #21-Ubuntu SMP Tue Apr 24 06:16:1=
5 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
```


--=20
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2069



--===============1863389327==
Content-Type: text/html
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="attachment.html"
MIME-Version: 1.0

PGgzPkRlc2NyaXB0aW9uPC9oMz4KPHA+PHN0cm9uZz5JIGRpZDo8L3N0cm9uZz48YnI+CkkgaGF2
ZSBpbnN0YWxsZWQga2FtYWlsaW8gNS4yLjQgaW4gdWJ1bnR1IHN5c3RlbSBhbmQgY29uZmlndXJl
ZCBrYW1haWxpbyBzY3JpcHQgdG8gY29ubmVjdCB3aXRoIGVybGFuZyBub2RlIGFsc28gaXQgc2Vu
ZHMgcnBjIHJlcXVlc3QgdG8gZXJsYW5nIG5vZGUgYW5kIGdldCByZXNwb25zZSB3aGljaCBJIGhh
dmUgcHJpbnRlZCBpbiBhIGxvZ3MuPGJyPgouLi48YnI+CiRlcmxfbGlzdChMKSA9ICJ0d28iOzxi
cj4KJGVybF9saXN0KEwpID0gIm9uZSI7PGJyPgp4bG9nKCJMX0FMRVJUIiwiQmVmb3JlIFJQQzog
JGVybF9saXN0KGFyZ3M9Jmd0O2Zvcm1hdClcbiIpOzxicj4KZXJsX3JwYygiZXJsYW5nIiwgImxp
c3RfdG9fdHVwbGUiLCAiJGVybF9saXN0KGFyZ3MpIiwgIiRlcmxfeGJ1ZmYocmVwbCkiKTs8YnI+
Cnhsb2dsKCJMX0FMRVJUIiwidHlwZShyZXBseSk6ICRlcmxfeGJ1ZmYocmVwbD0mZ3Q7dHlwZSks
IGZvcm1hdChyZXBsKTogJGVybF94YnVmZihyZXBsPSZndDtmb3JtYXQpXG4iKTs8YnI+Ci4uLjwv
cD4KPHA+PHN0cm9uZz5FeHBlY3RlZCB0byBoYXBwZW46PC9zdHJvbmc+PGJyPgpSZXNwb25zZSBz
aG91bGQgYmUgZ2V0IGZyb20gZXJsYW5nIG5vZGUgYWdhaW5zdCBSUEMgcmVxdWVzdCBhbmQgaXQg
c2hvdWxkIGJlIHN0b3JlIGluIGVybF94YnVmZi4gQW5kIGl0IHNob3VsZCBiZSBwcmludCBsb2dz
IGxpa2U6IEFMRVJUOiAmbHQ7c2NyaXB0Jmd0OzogNTAxOnR5cGUocmVwbHkpOiB0dXBsZSwgZm9y
bWF0KHJlcGwpOiB7Im9uZSIsICJ0d28ifTwvcD4KPHA+PHN0cm9uZz5BY3R1YWxseSBoYXBwZW5l
ZDo8L3N0cm9uZz48YnI+ClJlc3BvbnNlIGlzIGdldHRpbmcgcHJvcGVybHkuIGJ1dCB0aGUgcHJv
YmxlbSBpcyB0byBzdG9yZSBkYXRhIGludG8geGJ1ZmYgc28gaXQgcHJpbnQgbG9ncyBsaWtlOiBB
TEVSVDogJmx0O3NjcmlwdCZndDs6IDUwMTp0eXBlKHJlcGx5KTogLCBmb3JtYXQocmVwbCk6IDwv
cD4KPGgzPlRyb3VibGVzaG9vdGluZzwvaDM+CjxwPkkgaGF2ZSBjaGVjayB3aXRoIG9sZGVyIHZl
cnNpb24gNC4zLjAgYW5kIGl0IHdvcmtzIGZpbmUuIGdldHRpbmcgdGhpcyBwcm9ibGVtIGFmdGVy
IDUuMi4wIHZlcnNpb248YnI+ClNvLCBJIGhhdmUgY29tcGFyZSA1LjIuMCBhbmQgNC4zLjAgdmVy
c2lvbiBjb2RlLiBJIGZvdW5kIG9uZSBjaGFuZ2UgaW4gcHZfeGJ1ZmYuYyBmaWxlIHdoaWNoIGlz
IGNyZWF0aW5nIGEgcHJvYmxlbSB3aGVuIEkgcmV2ZXJ0IHRoZSBjaGFuZ2UgdGhlbiBpdCB3b3Jr
cyBmaW5lLjwvcD4KPGg0PlJlcHJvZHVjdGlvbjwvaDQ+CjxwPkl0IHdpbGwgYmUgcmVwcm9kdWNp
YmxlIGV2ZXJ5dGltZS4gSnVzdCBzZW5kcyBycGMgcmVxdWVzdCBhbmQgZ2V0IHRoZSB2YWx1ZSBp
biB4YnVmZi4gY2hlY2sgeGJ1ZmYgdmFsdWU8L3A+CjxoND5Mb2cgTWVzc2FnZXM8L2g0Pgo8cD5Q
cm9ibGVtIGxvZ3M6PC9wPgo8cHJlPjxjb2RlPkFMRVJUOiAmbHQ7c2NyaXB0Jmd0OzogQmVmb3Jl
IFJQQzogW1sib25lIiwgInR3byJdXQpXQVJOSU5HOiAmbHQ7Y29yZSZndDsgW2NvcmUvbWVtL3Ff
bWFsbG9jLmM6NDgwXTogcW1fZnJlZSgpOiBXQVJOSU5HOiBmcmVlKDApIGNhbGxlZCBmcm9tIGVy
bGFuZzogcHZfeGJ1ZmYuYzogeGF2cF9kZWNvZGUoMTIwOSkKQUxFUlQ6ICZsdDtzY3JpcHQmZ3Q7
OiA1MDE6dHlwZShyZXBseSk6ICZsdDtudWxsJmd0OywgZm9ybWF0KHJlcGwpOiAmbHQ7bnVsbCZn
dDsKPC9jb2RlPjwvcHJlPgo8cD5sb2dzIGFmdGVyIGNoYW5nZTo8L3A+CjxwcmU+PGNvZGU+QUxF
UlQ6ICZsdDtzY3JpcHQmZ3Q7OiBCZWZvcmUgUlBDOiBbWyJvbmUiLCAidHdvIl1dCldBUk5JTkc6
ICZsdDtjb3JlJmd0OyBbY29yZS9tZW0vcV9tYWxsb2MuYzo0ODBdOiBxbV9mcmVlKCk6IFdBUk5J
Tkc6IGZyZWUoMCkgY2FsbGVkIGZyb20gZXJsYW5nOiBwdl94YnVmZi5jOiB4YXZwX2RlY29kZSgx
MjA5KQpBTEVSVDogJmx0O3NjcmlwdCZndDs6IDUwMTp0eXBlKHJlcGx5KTogdHVwbGUsIGZvcm1h
dChyZXBsKTogeyJvbmUiLCAidHdvIn0KPC9jb2RlPjwvcHJlPgo8aDM+UG9zc2libGUgU29sdXRp
b25zPC9oMz4KPHA+cmV2ZXJ0IHRoZSA8YSBjbGFzcz0iY29tbWl0LWxpbmsiIGRhdGEtaG92ZXJj
YXJkLXR5cGU9ImNvbW1pdCIgZGF0YS1ob3ZlcmNhcmQtdXJsPSJodHRwczovL2dpdGh1Yi5jb20v
a2FtYWlsaW8va2FtYWlsaW8vY29tbWl0LzY1NDJhMzY3NjM4ZWM1ZjhhMmI2Mjk5NGY4YzMwOWU3
YTI1MzM5NjIvaG92ZXJjYXJkIiBocmVmPSJodHRwczovL2dpdGh1Yi5jb20va2FtYWlsaW8va2Ft
YWlsaW8vY29tbWl0LzY1NDJhMzY3NjM4ZWM1ZjhhMmI2Mjk5NGY4YzMwOWU3YTI1MzM5NjIiPjx0
dD42NTQyYTM2PC90dD48L2E+IGNvbW1pdC4gSXQgd2lsbCB3b3JrIGZpbmU8L3A+CjxoMz5BZGRp
dGlvbmFsIEluZm9ybWF0aW9uPC9oMz4KPHByZT48Y29kZT52ZXJzaW9uOiBrYW1haWxpbyA1LjIu
NCAoeDg2XzY0L2xpbnV4KSA3NTk4NjcKZmxhZ3M6IFNUQVRTOiBPZmYsIFVTRV9UQ1AsIFVTRV9U
TFMsIFVTRV9TQ1RQLCBUTFNfSE9PS1MsIFVTRV9SQVdfU09DS1MsIERJU0FCTEVfTkFHTEUsIFVT
RV9NQ0FTVCwgRE5TX0lQX0hBQ0ssIFNITV9NRU0sIFNITV9NTUFQLCBQS0dfTUFMTE9DLCBRX01B
TExPQywgRl9NQUxMT0MsIFRMU0ZfTUFMTE9DLCBEQkdfU1JfTUVNT1JZLCBVU0VfRlVURVgsIEZB
U1RfTE9DSy1BREFQVElWRV9XQUlULCBVU0VfRE5TX0NBQ0hFLCBVU0VfRE5TX0ZBSUxPVkVSLCBV
U0VfTkFQVFIsIFVTRV9EU1RfQkxBQ0tMSVNULCBIQVZFX1JFU09MVl9SRVMKQURBUFRJVkVfV0FJ
VF9MT09QUz0xMDI0LCBNQVhfUkVDVl9CVUZGRVJfU0laRSAyNjIxNDQgTUFYX1VSSV9TSVpFIDEw
MjQsIEJVRl9TSVpFIDY1NTM1LCBERUZBVUxUIFBLR19TSVpFIDhNQgpwb2xsIG1ldGhvZCBzdXBw
b3J0OiBwb2xsLCBlcG9sbF9sdCwgZXBvbGxfZXQsIHNpZ2lvX3J0LCBzZWxlY3QuCmlkOiA3NTk4
NjcgCmNvbXBpbGVkIG9uIDExOjUyOjU1IFNlcCAgNCAyMDE5IHdpdGggZ2NjIDcuNC4wCgo8L2Nv
ZGU+PC9wcmU+Cjx1bD4KPGxpPjxzdHJvbmc+T3BlcmF0aW5nIFN5c3RlbTwvc3Ryb25nPjo8L2xp
Pgo8L3VsPgo8cHJlPjxjb2RlPkRpc3RyaWJ1dG9yIElEOglVYnVudHUKRGVzY3JpcHRpb246CVVi
dW50dSAxOC4wNCBMVFMKUmVsZWFzZToJMTguMDQKQ29kZW5hbWU6CWJpb25pYwoKTGludXggaG9z
dG5hbWUuZG9tYWluLmNvbSA0LjE1LjAtMjAtZ2VuZXJpYyAjMjEtVWJ1bnR1IFNNUCBUdWUgQXBy
IDI0IDA2OjE2OjE1IFVUQyAyMDE4IHg4Nl82NCB4ODZfNjQgeDg2XzY0IEdOVS9MaW51eAo8L2Nv
ZGU+PC9wcmU+Cgo8cCBzdHlsZT0iZm9udC1zaXplOnNtYWxsOy13ZWJraXQtdGV4dC1zaXplLWFk
anVzdDpub25lO2NvbG9yOiM2NjY7Ij4mbWRhc2g7PGJyIC8+WW91IGFyZSByZWNlaXZpbmcgdGhp
cyBiZWNhdXNlIHlvdSBhcmUgc3Vic2NyaWJlZCB0byB0aGlzIHRocmVhZC48YnIgLz5SZXBseSB0
byB0aGlzIGVtYWlsIGRpcmVjdGx5LCA8YSBocmVmPSJodHRwczovL2dpdGh1Yi5jb20va2FtYWls
aW8va2FtYWlsaW8vaXNzdWVzLzIwNjk/ZW1haWxfc291cmNlPW5vdGlmaWNhdGlvbnMmYW1wO2Vt
YWlsX3Rva2VuPUFCTzdVWkpHQVdXWkxRM0JJUkxYNUlEUUtNU1pOQTVDTkZTTTRJWUhPVFIyWVkz
UE5WV1dLM1RVTDUySFM0REZVVkVYRzQzVk1XVkdHMzNOTlZTVzQ1QzdORlNNNEhNS1RISFEiPnZp
ZXcgaXQgb24gR2l0SHViPC9hPiwgb3IgPGEgaHJlZj0iaHR0cHM6Ly9naXRodWIuY29tL25vdGlm
aWNhdGlvbnMvdW5zdWJzY3JpYmUtYXV0aC9BQk83VVpQM0E3NlNXQzJJRFJRQk9NVFFLTVNaTkFO
Q05GU000SVlIT1RSUSI+bXV0ZSB0aGUgdGhyZWFkPC9hPi48aW1nIHNyYz0iaHR0cHM6Ly9naXRo
dWIuY29tL25vdGlmaWNhdGlvbnMvYmVhY29uL0FCTzdVWk9GR1pZSzZBTFhXM0kzMlEzUUtNU1pO
QTVDTkZTTTRJWUhPVFIyWVkzUE5WV1dLM1RVTDUySFM0REZVVkVYRzQzVk1XVkdHMzNOTlZTVzQ1
QzdORlNNNEhNS1RISFEuZ2lmIiBoZWlnaHQ9IjEiIHdpZHRoPSIxIiBhbHQ9IiIgLz48L3A+Cjxz
Y3JpcHQgdHlwZT0iYXBwbGljYXRpb24vbGQranNvbiI+Wwp7CiJAY29udGV4dCI6ICJodHRwOi8v
c2NoZW1hLm9yZyIsCiJAdHlwZSI6ICJFbWFpbE1lc3NhZ2UiLAoicG90ZW50aWFsQWN0aW9uIjog
ewoiQHR5cGUiOiAiVmlld0FjdGlvbiIsCiJ0YXJnZXQiOiAiaHR0cHM6Ly9naXRodWIuY29tL2th
bWFpbGlvL2thbWFpbGlvL2lzc3Vlcy8yMDY5P2VtYWlsX3NvdXJjZT1ub3RpZmljYXRpb25zXHUw
MDI2ZW1haWxfdG9rZW49QUJPN1VaSkdBV1daTFEzQklSTFg1SURRS01TWk5BNUNORlNNNElZSE9U
UjJZWTNQTlZXV0szVFVMNTJIUzRERlVWRVhHNDNWTVdWR0czM05OVlNXNDVDN05GU000SE1LVEhI
USIsCiJ1cmwiOiAiaHR0cHM6Ly9naXRodWIuY29tL2thbWFpbGlvL2thbWFpbGlvL2lzc3Vlcy8y
MDY5P2VtYWlsX3NvdXJjZT1ub3RpZmljYXRpb25zXHUwMDI2ZW1haWxfdG9rZW49QUJPN1VaSkdB
V1daTFEzQklSTFg1SURRS01TWk5BNUNORlNNNElZSE9UUjJZWTNQTlZXV0szVFVMNTJIUzRERlVW
RVhHNDNWTVdWR0czM05OVlNXNDVDN05GU000SE1LVEhIUSIsCiJuYW1lIjogIlZpZXcgSXNzdWUi
Cn0sCiJkZXNjcmlwdGlvbiI6ICJWaWV3IHRoaXMgSXNzdWUgb24gR2l0SHViIiwKInB1Ymxpc2hl
ciI6IHsKIkB0eXBlIjogIk9yZ2FuaXphdGlvbiIsCiJuYW1lIjogIkdpdEh1YiIsCiJ1cmwiOiAi
aHR0cHM6Ly9naXRodWIuY29tIgp9Cn0KXTwvc2NyaXB0Pg==

--===============1863389327==--