Calling av_free with NULL is a no-op so this check is not needed.
Unified · Split
| @@ -228,8 +228,8 @@ end: · 227 unmodified lines | ||
| 228 | 228 | av_bprint_finalize(&key_bp, NULL); |
| 229 | 229 | BIO_free(cert_b); |
| 230 | 230 | av_bprint_finalize(&cert_bp, NULL); |
| 231 | − if (key_tem) av_free(key_tem); | |
| 232 | − if (cert_tem) av_free(cert_tem); | |
| 231 | + av_free(key_tem); | |
| 232 | + av_free(cert_tem); | |
| 233 | 233 | return ret; |
| 234 | 234 | } |
| 235 | 235 | |
| @@ -392,8 +392,8 @@ int ff_ssl_gen_key_cert(char *key_buf, size_t key_sz, char *cert_buf, size_t cer · 156 unmodified lines | ||
| 392 | 392 | snprintf(key_buf, key_sz, "%s", key_tem); |
| 393 | 393 | snprintf(cert_buf, cert_sz, "%s", cert_tem); |
| 394 | 394 | |
| 395 | − if (key_tem) av_free(key_tem); | |
| 396 | − if (cert_tem) av_free(cert_tem); | |
| 395 | + av_free(key_tem); | |
| 396 | + av_free(cert_tem); | |
| 397 | 397 | error: |
| 398 | 398 | return ret; |
| 399 | 399 | } |