Merge pull request #433 from xrmx/coverity2810-2

More Coverity fixes
This commit is contained in:
unbit
2013-10-28 14:13:25 -07:00
4 changed files with 9 additions and 3 deletions
+5
View File
@@ -114,6 +114,11 @@ int uwsgi_master_fifo_manage(int fd) {
return 0;
}
if ((int)cmd < 0) {
uwsgi_error("invalid cmd read in uwsgi_master_fifo_manage");
exit(1);
}
if (uwsgi_fifo_table[(int) cmd]) {
uwsgi_fifo_table[(int) cmd](0);
}
+1 -1
View File
@@ -1222,7 +1222,7 @@ void uwsgi_linux_ksm_map(void) {
}
// we now have areas
if (uwsgi.ksm_mappings_last_size == 0 || uwsgi.ksm_mappings_current_size == 0 || uwsgi.ksm_mappings_current_size != uwsgi.ksm_mappings_last_size) {
if (uwsgi.ksm_mappings_last_size == 0 || uwsgi.ksm_mappings_current_size != uwsgi.ksm_mappings_last_size) {
dirty = 1;
}
else {
+2 -1
View File
@@ -291,7 +291,8 @@ request:
}
end:
Py_DECREF(greenlet_switch);
if (greenlet_switch)
Py_DECREF(greenlet_switch);
end2:
Py_DECREF(current_greenlet);
+1 -1
View File
@@ -81,7 +81,7 @@ static int uwsgi_routing_func_hash(struct wsgi_request *wsgi_req, struct uwsgi_r
vallen = urhc->items_len;
}
// last item
else {
else if (value != NULL) {
vallen = (urhc->items + urhc->items_len) - value;
}
}