[Home]

Summary:ASTERISK-12362: [patch] Fix 'core show sysinfo' command, it is returning all 0s.
Reporter:Eliel Sardanons (eliel)Labels:
Date Opened:2008-07-10 14:49:57Date Closed:2008-07-16 16:46:25
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20080710_issue13057_sysinfo_memunit.diff
( 1) asterisk.c.patch
( 2) func_sysinfo.patch
Description:sysinfo() for kernels > 2.3.23 is returning all values in mem_unit units, so to get the number of bytes we must multiply it by mem_unit and not divide it.
Comments:By: Jeff Peeler (jpeeler) 2008-07-10 18:47:16

Same exact code is used in func_sysinfo. This is mainly a reminder to myself.
By: Eliel Sardanons (eliel) 2008-07-10 20:32:19

Thank you Jeff, patch uploaded.
By: Digium Subversion (svnbot) 2008-07-16 16:16:31

Repository: asterisk
Revision: 131445

U   trunk/main/asterisk.c

------------------------------------------------------------------------
r131445 | bbryant | 2008-07-16 16:16:30 -0500 (Wed, 16 Jul 2008) | 9 lines

Fixes an issue with "core show sysinfo" that used the wrong operator to
calculate the number of bytes from a sysinfo structure.
unsigned long.

(closes issue ASTERISK-12362)
Reported by: eliel
Patches:
     asterisk.c.patch uploaded by eliel (license 64)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=131445
By: Digium Subversion (svnbot) 2008-07-16 16:46:25

Repository: asterisk
Revision: 131484

U   trunk/funcs/func_sysinfo.c

------------------------------------------------------------------------
r131484 | bbryant | 2008-07-16 16:46:24 -0500 (Wed, 16 Jul 2008) | 4 lines

Fixes sysinfo operator issue also fixed elsewhere in r131445.

(issue ASTERISK-12362)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=131484