diff options
| author | Gustavo F. Padovan <gustavo@padovan.org> | 2010-06-05 07:14:29 -0300 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@nokia.com> | 2010-06-08 15:57:25 +0800 |
| commit | a554d3a656f0b0e3a3d82b68ab61b89f1b876660 (patch) | |
| tree | 1c293f67a8ca6db7f985b074bd504b7bcc27ca66 /src/sdpd-request.c | |
| parent | 502bfaf2bccfac0826a3802a5e95bafaf976d981 (diff) | |
| download | bluez-a554d3a656f0b0e3a3d82b68ab61b89f1b876660.tar.gz | |
Fix redundant null check on calling free()
Issues found by smatch static check: http://smatch.sourceforge.net/
Diffstat (limited to 'src/sdpd-request.c')
| -rw-r--r-- | src/sdpd-request.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/src/sdpd-request.c b/src/sdpd-request.c index 8c88d6ead..d56ffc25b 100644 --- a/src/sdpd-request.c +++ b/src/sdpd-request.c @@ -526,8 +526,7 @@ static int service_search_req(sdp_req_t *req, sdp_buf_t *buf) } done: - if (cstate) - free(cstate); + free(cstate); if (pattern) sdp_list_free(pattern, free); @@ -745,8 +744,7 @@ static int service_attr_req(sdp_req_t *req, sdp_buf_t *buf) buf->buf_size += sizeof(uint16_t); done: - if (cstate) - free(cstate); + free(cstate); if (seq) sdp_list_free(seq, free); if (status) @@ -929,10 +927,8 @@ static int service_search_attr_req(sdp_req_t *req, sdp_buf_t *buf) } done: - if (cstate) - free(cstate); - if (tmpbuf.data) - free(tmpbuf.data); + free(cstate); + free(tmpbuf.data); if (pattern) sdp_list_free(pattern, free); if (seq) |