player: remove extremely obscure undefined behavior

Apparently the value of a pointer is "indeterminate" after a free()
call, even if you never dereference the pointer after the free. Since
talloc_free() calls free(), this applies here.

1 files changed, 2 insertions, 1 deletions

diff --git a/player/video.c b/player/video.c
index 59acce0a19..1bb06ad0cc 100644
--- a/player/video.c
+++ b/player/video.c
@@ -326,9 +326,10 @@ static int decode_image(struct MPContext *mpctx)
                          1 : check_framedrop(mpctx, -1);
     d_video->waiting_decoded_mpi =
         video_decode(d_video, pkt, framedrop_type);
+    bool had_packet = !!pkt;
     talloc_free(pkt);
 
-    return !!pkt;
+    return had_packet;
 }