diff options
| author | Hendrik Leppkes <h.leppkes@gmail.com> | 2015-12-12 18:56:05 +0300 |
|---|---|---|
| committer | Hendrik Leppkes <h.leppkes@gmail.com> | 2016-01-04 14:16:15 +0300 |
| commit | 6d5be0514e1fe6dd3a705960cc4e90b5ed8fb777 (patch) | |
| tree | 61a558761bf81eb9242659c91cd77aa4da2e34b0 | |
| parent | 1f4805da165f9c30e1c0d1f9de8db183305ca846 (diff) | |
avformat/httpauth: change order of qop parametersmpc-hc-1.7.10-3
This fixes a bug when interacting with some (broken) HTTP servers which
expect the last entry to be a quoted string.
Also remove the quoting of the qop parameter in the process, it violates
the specification.
| -rw-r--r-- | libavformat/httpauth.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libavformat/httpauth.c b/libavformat/httpauth.c index 2d42ab2190..61b48a5167 100644 --- a/libavformat/httpauth.c +++ b/libavformat/httpauth.c @@ -232,9 +232,9 @@ static char *make_digest_auth(HTTPAuthState *state, const char *username, if (digest->opaque[0]) av_strlcatf(authstr, len, ", opaque=\"%s\"", digest->opaque); if (digest->qop[0]) { - av_strlcatf(authstr, len, ", qop=\"%s\"", digest->qop); - av_strlcatf(authstr, len, ", cnonce=\"%s\"", cnonce); + av_strlcatf(authstr, len, ", qop=%s", digest->qop); av_strlcatf(authstr, len, ", nc=%s", nc); + av_strlcatf(authstr, len, ", cnonce=\"%s\"", cnonce); } av_strlcatf(authstr, len, "\r\n"); |