Merge pull request #359 from signalwire/apr_util
[apr-util] Fix issues reported by scan-build.
This commit is contained in:
commit
944041fb46
|
@ -180,7 +180,9 @@ static TSort *tsort(TSort *pData,int nItems)
|
|||
break;
|
||||
}
|
||||
}
|
||||
pTail->pNext=NULL; /* unfudge the tail */
|
||||
if (pTail) {
|
||||
pTail->pNext = NULL; /* unfudge the tail */
|
||||
}
|
||||
return pHead;
|
||||
}
|
||||
|
||||
|
|
|
@ -222,7 +222,7 @@ APU_DECLARE(apr_status_t) apr_queue_trypush(apr_queue_t *queue, void *data)
|
|||
}
|
||||
|
||||
if (apr_queue_full(queue)) {
|
||||
rv = apr_thread_mutex_unlock(queue->one_big_mutex);
|
||||
apr_thread_mutex_unlock(queue->one_big_mutex);
|
||||
return APR_EAGAIN;
|
||||
}
|
||||
|
||||
|
@ -397,7 +397,7 @@ APU_DECLARE(apr_status_t) apr_queue_trypop(apr_queue_t *queue, void **data)
|
|||
}
|
||||
|
||||
if (apr_queue_empty(queue)) {
|
||||
rv = apr_thread_mutex_unlock(queue->one_big_mutex);
|
||||
apr_thread_mutex_unlock(queue->one_big_mutex);
|
||||
return APR_EAGAIN;
|
||||
}
|
||||
|
||||
|
|
|
@ -82,7 +82,7 @@ static int find_prefix(apr_xml_parser *parser, const char *prefix)
|
|||
** prefix.
|
||||
*/
|
||||
for (; elem; elem = elem->parent) {
|
||||
apr_xml_ns_scope *ns_scope = elem->ns_scope;
|
||||
apr_xml_ns_scope *ns_scope;
|
||||
|
||||
for (ns_scope = elem->ns_scope; ns_scope; ns_scope = ns_scope->next) {
|
||||
if (strcmp(prefix, ns_scope->prefix) == 0) {
|
||||
|
|
Loading…
Reference in New Issue