luo980/curl
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

rtsp: skip NULL assigns after Curl_safefree()

Browse Source 
... since this is a macro that assigns NULL itself. Pointed out by PVS.

Ref: #10929
Closes #10946
This commit is contained in:
Daniel Stenberg 2023-04-13 09:36:57 +02:00
parent d50045509c
commit aa3ee0a13a
No known key found for this signature in database
GPG Key ID: 5CC908FDB71E12C2
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/rtsp.c
View File

@ -376,7 +376,6 @@ static CURLcode rtsp_do(struct Curl_easy *data, bool *done)
if(Curl_checkheaders(data, STRCONST("User-Agent")) && if(Curl_checkheaders(data, STRCONST("User-Agent")) &&
data->state.aptr.uagent) { data->state.aptr.uagent) {
Curl_safefree(data->state.aptr.uagent); Curl_safefree(data->state.aptr.uagent);
data->state.aptr.uagent = NULL;
} }
else if(!Curl_checkheaders(data, STRCONST("User-Agent")) && else if(!Curl_checkheaders(data, STRCONST("User-Agent")) &&
data->set.str[STRING_USERAGENT]) { data->set.str[STRING_USERAGENT]) {
@ -396,8 +395,6 @@ static CURLcode rtsp_do(struct Curl_easy *data, bool *done)
Curl_safefree(data->state.aptr.ref); Curl_safefree(data->state.aptr.ref);
if(data->state.referer && !Curl_checkheaders(data, STRCONST("Referer"))) if(data->state.referer && !Curl_checkheaders(data, STRCONST("Referer")))
data->state.aptr.ref = aprintf("Referer: %s\r\n", data->state.referer); data->state.aptr.ref = aprintf("Referer: %s\r\n", data->state.referer);
else
data->state.aptr.ref = NULL;
p_referrer = data->state.aptr.ref; p_referrer = data->state.aptr.ref;
@ -478,7 +475,6 @@ static CURLcode rtsp_do(struct Curl_easy *data, bool *done)
* with basic and digest, it will be freed anyway by the next request * with basic and digest, it will be freed anyway by the next request
*/ */
Curl_safefree(data->state.aptr.userpwd); Curl_safefree(data->state.aptr.userpwd);
data->state.aptr.userpwd = NULL;
if(result) if(result)
return result; return result;