am7xxx-play: fix a possible case of use before initialization
authorAntonio Ospite <ao2@ao2.it>
Sat, 22 Nov 2014 22:58:36 +0000 (23:58 +0100)
committerAntonio Ospite <ao2@ao2.it>
Sat, 22 Nov 2014 22:58:36 +0000 (23:58 +0100)
The got_packet variable is assigned inside an internal "if" of the while
loop, but it is also used in the checks at the end of the while loop,
and it is possible to get to this part before having ever assigned the
variable, so in order to behave correctly initialize the variable before
the loop starts.

examples/am7xxx-play.c

index 2fa615a..5b2887d 100644 (file)
@@ -354,6 +354,7 @@ static int am7xxx_play(const char *input_format_string,
                goto cleanup_out_buf;
        }
 
                goto cleanup_out_buf;
        }
 
+       got_packet = 0;
        while (run) {
                /* read packet */
                ret = av_read_frame(input_ctx.format_ctx, &in_packet);
        while (run) {
                /* read packet */
                ret = av_read_frame(input_ctx.format_ctx, &in_packet);