diff options
| author | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-10-27 19:54:07 -0400 |
|---|---|---|
| committer | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-11-17 16:30:15 -0500 |
| commit | 3fb32ae2cf03e4f33f14a5080eb7924a8536cd28 (patch) | |
| tree | f557059b1bb3bcc9b1ac3dc54d4396fe461ec5c6 | |
| parent | a5034b324cad4c29d47ef285a30b0705e6eb0384 (diff) | |
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@niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
| -rw-r--r-- | libavformat/rtpdec_mpa_robust.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/rtpdec_mpa_robust.c b/libavformat/rtpdec_mpa_robust.c index 6442a427d0..86c8958d5c 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"); |