fix minor bug printing unknown aiff subchunks (SF bug #1267476: https://sourceforge...
authorJosh Coalson <jcoalson@users.sourceforce.net>
Mon, 29 Aug 2005 04:30:20 +0000 (04:30 +0000)
committerJosh Coalson <jcoalson@users.sourceforce.net>
Mon, 29 Aug 2005 04:30:20 +0000 (04:30 +0000)
doc/html/changelog.html
src/flac/encode.c

index 7f64f8c..93cb415 100644 (file)
@@ -88,6 +88,7 @@
                                        <li>Fixed a bug in cuesheet parsing where it would return an error if the last line of the cuesheet did not end with a newline.</li>
                                        <li>Fixed a bug with --sector-align where appended samples were not always totally silent (<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=1237707&group_id=13478&atid=113478">SF #1237707</a>).</li>
                                        <li>Fixed bugs with --sector-align and raw input files.</li>
+                                       <li>Fixed a bug printing out unknown AIFF subchunk names (<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=1267476&group_id=13478&atid=113478">SF #1267476</a>).</li>
                                </ul>
                        </li>
                        <li>
index 61f076c..a964bec 100644 (file)
@@ -194,7 +194,7 @@ int flac__encode_aif(FILE *infile, long infilesize, const char *infilename, cons
 
        while(1) {
                size_t c= 0U;
-               char chunk_id[4];
+               char chunk_id[5] = { '\0', '\0', '\0', '\0', '\0' }; /* one extra byte for terminating NUL so we can also treat it like a C string */
 
                /* chunk identifier; really conservative about behavior of fread() and feof() */
                if(feof(infile) || ((c= fread(chunk_id, 1U, 4U, infile)), c==0U && feof(infile)))