[FFmpeg-cvslog] avformat/rtpdec_mpa_robust: change assignment to inequality test in conditional
Ganesh Ajjanagadde
git at videolan.org
Tue Nov 17 22:31:06 CET 2015
ffmpeg | branch: master | Ganesh Ajjanagadde <gajjanagadde at gmail.com> | Tue Oct 27 19:54:07 2015 -0400| [3fb32ae2cf03e4f33f14a5080eb7924a8536cd28] | committer: Ganesh Ajjanagadde
avformat/rtpdec_mpa_robust: change assignment to inequality test in conditional
In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely
typo originally.
Found by enabling -Wparentheses on clang 3.7 and running a manual audit.
To the best of my knowledge, no such instances remain.
Reviewed-by: Michael Niedermayer <michael at niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3fb32ae2cf03e4f33f14a5080eb7924a8536cd28
---
libavformat/rtpdec_mpa_robust.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/rtpdec_mpa_robust.c b/libavformat/rtpdec_mpa_robust.c
index 6442a42..86c8958 100644
--- a/libavformat/rtpdec_mpa_robust.c
+++ b/libavformat/rtpdec_mpa_robust.c
@@ -166,7 +166,7 @@ static int mpa_robust_parse_packet(AVFormatContext *ctx, PayloadContext *data,
"Received packet without a start fragment; dropping.\n");
return AVERROR(EAGAIN);
}
- if (adu_size = data->adu_size ||
+ if (adu_size != data->adu_size ||
data->timestamp != *timestamp) {
ffio_free_dyn_buf(&data->fragment);
av_log(ctx, AV_LOG_ERROR, "Invalid packet received\n");
More information about the ffmpeg-cvslog
mailing list