diff options
| author | Martin Storsjö <martin@martin.st> | 2015-02-24 12:32:17 +0200 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2015-02-24 23:07:22 +0200 |
| commit | ec96a89c3e507cf0fb1f2b159b28a53f2bad9a74 (patch) | |
| tree | 71cf3f51975e8e728dadb0f3d939d79e73956e29 /libavformat/rtpdec_amr.c | |
| parent | ac0e54fda9305cc7d149007f5b512bb8619f7c78 (diff) | |
rtpdec: Don't pass non-const pointers to fmtp attribute parsing functions
This makes it clear that the individual parsing functions can't
touch the parsed out value.
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'libavformat/rtpdec_amr.c')
| -rw-r--r-- | libavformat/rtpdec_amr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavformat/rtpdec_amr.c b/libavformat/rtpdec_amr.c index 897873b63f..8216a5b911 100644 --- a/libavformat/rtpdec_amr.c +++ b/libavformat/rtpdec_amr.c @@ -137,7 +137,7 @@ static int amr_handle_packet(AVFormatContext *ctx, PayloadContext *data, static int amr_parse_fmtp(AVFormatContext *s, AVStream *stream, PayloadContext *data, - char *attr, char *value) + const char *attr, const char *value) { /* Some AMR SDP configurations contain "octet-align", without * the trailing =1. Therefore, if the value is empty, @@ -146,7 +146,7 @@ static int amr_parse_fmtp(AVFormatContext *s, if (!strcmp(value, "")) { av_log(s, AV_LOG_WARNING, "AMR fmtp attribute %s had " "nonstandard empty value\n", attr); - strcpy(value, "1"); + value = "1"; } if (!strcmp(attr, "octet-align")) data->octet_align = atoi(value); |