[Home]

Summary:ASTERISK-00877: app_festival, possible to exceed string length in code MD5Hex
Reporter:woofie (woofie)Labels:
Date Opened:2004-01-19 10:01:41.000-0600Date Closed:2008-01-15 14:42:00.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:MD5Hex string variable (line 266) allocates 32 bytes
Routine starting line 353 can (not always) concatenate
2 bytes to array 16 times.  If string is full 32 bytes, no room for final null.  Recommend MD5Hex at least 33 bytes.

Null overwrite could cause undetermined behavior.

New to asterisk and bug fix process.  Apologize for no diff patch on this.  But Thought info would be good anyways.
Comments:By: Brian West (bkw918) 2004-01-19 12:28:58.000-0600

attach a patch for that please. :)

By: woofie (woofie) 2004-01-20 23:01:44.000-0600

will do.

By: Mark Spencer (markster) 2004-01-22 15:56:32.000-0600

Fixed in CVS

By: Digium Subversion (svnbot) 2008-01-15 14:42:00.000-0600

Repository: asterisk
Revision: 2060

U   trunk/apps/app_festival.c

------------------------------------------------------------------------
r2060 | markster | 2008-01-15 14:41:59 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix MD5Hex size (bug ASTERISK-877)

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

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