Hi,
we are using kamailio with more than 4GB of shared memory. When using 'core.shmem' we are expierencing an integer overflow in the output.
The issue can be found here:
https://github.com/kamailio/kamailio/blob/1ddc27f199061025a6a43da3e8a1388fca...
I currently don't have the time to setup a build environment so I can't fix it myself.
cheers.
A sexta, 13/08/2021, 10:38, Julius Flohr ***@***.***> escreveu:
Hi,
we are using kamailio with more than 4GB of shared memory. When using 'core.shmem' we are expierencing an integer overflow in the output.
The issue can be found here:
https://github.com/kamailio/kamailio/blob/1ddc27f199061025a6a43da3e8a1388fca...
I currently don't have the time to setup a build environment so I can't fix it myself.
cheers.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kamailio/kamailio/issues/2824, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAY5TN2N7S27ZZPEG4VGALT4TRYHANCNFSM5CDHQ4OA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .
The rcp supported int and double types, the values being stored over an int field. I added a few more conversion types for long/long long variants, storing them over the double field. The standard jsonrpc defines anyhow on 'number' (double) as a type.
binrpc had a further hack for double fields, being serialized as integer. I changed to serialize as long long.
kamctl didn't need any change as it prints the numbers coming in the formatted json. kamcmd print the double values as long long if they have no decimals.
Overall, the current master should print properly the shm stats with values over 4G. If not, open a new issue.
There were many changes, practically the rpc framework got support for new data types, therefore won't be backported.
Closed #2824.
-
Daniel-Constantin Mierla -
José Seabra
-
Julius Flohr