[Home]

Summary:ASTERISK-07704: GSM codec: formats/msgsm.h: xmc[48] not initialized?!
Reporter:Ralf S. Engelschall (rse)Labels:
Date Opened:2006-09-09 12:53:43Date Closed:2011-06-07 14:08:28
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/CodecInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:On building Asterisk 1.2.12 I recognized the following GCC 4 warning:

../formats/msgsm.h:459: warning: 'xmc[48]' may be used uninitialized in this function

I then had a look into msgsm.h:459 and especially msgsm.h:565
and xmc[48] is really not initialized. I've to admit that I've
no clue whether it is correct or incorrect to now set xmc[48]
in this piece of code and if it is really missing whether
is leads a problem or not. But at least the Asterisk developers
should review this part and either fix it by setting xmc[48]
there or at least add a comment "/* xmc[48] intentionally not set */"
there in case someone else again stumbles over this issue.

Thanks.
Comments:By: Joshua C. Colp (jcolp) 2006-09-09 15:41:48

This isn't a real life threatening bug or anything and has already been solved in trunk.