[FFmpeg-devel] [PATCH] lavf/matroskadec: Ensure cues_end is initialized

ooe at st-andrews.ac.uk ooe at st-andrews.ac.uk
Tue Aug 26 15:33:56 CEST 2014


<div name="BB10" id="BB10_response_div" dir="auto" contenteditable="true"                                                                                             style="width:100%;background:"#ffffff"; font-size: initial;font-family:"Calibri","Slate Pro","sans-serif";color:#1f497d;"><br style="display:initial"></div>                                                                                                                                     <div name="BB10" id="response_div_spacer" dir="auto" contenteditable="true"                                                                                           style="width:100%;background:"#ffffff"; font-size: initial;font-family:"Calibri","Slate Pro","sans-serif";color:#1f497d;"><br style="display:initial"></div>                                                                                                                                     <div id="_signaturePlaceholder" name="BB10" dir="auto"  contenteditable="true"                                                                                      style="font-size: initial;font-family:"Calibri","Slate Pro","sans-serif";color:#1f497d;"></div>                                       <table id="_bb10TempSeparator" width="100%" style="background-color:white; border-spacing:0px;" contenteditable="false">                                               <tr><td></td><td id="_separatorInternal" rowspan=2 style="text-align:center">                                                                                            <span id="_bb10TempSeparatorText" style="background-color:white; color:#0073BC;font-size:smaller;font-family:"Slate Pro"">  Original Message  </span>      </td></tr>                                                                                                                                                               <tr> <td colspan=2><div style="border:none;border-top:solid #0073BC 1.0pt;">                                                                                           </td></tr></table>                                                                                                                                                 <table width="100%" style="background-color:white;border-spacing:0px;"> <tr><td id="_persistentHeaderContainer" colspan=2>                                              <div id="_persistentHeader" style="font-size: smaller;font-family:"Tahoma","BB Alpha Sans","Slate Pro","sans-serif";">  <div id=From ><b>From: </b>Reimar Döffinger</div><div id=Sent ><b>Sent: </b>Tuesday, 26 August 2014 08:31</div><div id=To ><b>To: </b>FFmpeg development discussions and patches</div><div id=ReplyTo ><b>Reply To: </b>FFmpeg development discussions and patches</div><div id=Subject ><b>Subject: </b>Re: [FFmpeg-devel] [PATCH] lavf/matroskadec: Ensure cues_end is initialized</div></div></td></tr></table><div id="_persistentHeaderEnd" style="border:none;border-top:solid #babcd1 1pt;"></div><br><div name="BB10" dir="auto" id="_originalContent"><br/><br/>On 25.08.2014, at 18:15, Vignesh Venkatasubramanian <vigneshv at google.com> wrote:<br/><br/>>     avio_seek(matroska->ctx->pb, before_pos, SEEK_SET);<br/>> +    if (cues_start == -1 || cues_end == -1) return -1;<br/><br/>Shouldn't this use something more descriptive than -1?<br/>_______________________________________________<br/>ffmpeg-devel mailing list<br/>ffmpeg-devel at ffmpeg.org<br/>http://ffmpeg.org/mailman/listinfo/ffmpeg-devel<br/>


More information about the ffmpeg-devel mailing list