Skip to content

Commit 9b5ce37

Browse files
committed
Use more defensive approach in the switch statement.
Change-Id: I374fac5f83dae3a63f21b9177223c6175bec7300 Signed-off-by: Piotr Sikora <[email protected]>
1 parent 921d937 commit 9b5ce37

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

ngx_cache_purge_module.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -689,20 +689,20 @@ ngx_http_cache_purge_handler(ngx_http_request_t *r)
689689
# endif
690690

691691
switch (ngx_http_file_cache_purge(r)) {
692+
case NGX_OK:
693+
r->write_event_handler = ngx_http_request_empty_handler;
694+
ngx_http_finalize_request(r, ngx_http_cache_purge_send_response(r));
695+
return;
692696
case NGX_DECLINED:
693697
ngx_http_finalize_request(r, NGX_HTTP_NOT_FOUND);
694698
return;
695-
case NGX_ERROR:
696-
ngx_http_finalize_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
697-
return;
698699
# if (NGX_HAVE_FILE_AIO)
699700
case NGX_AGAIN:
700701
r->write_event_handler = ngx_http_cache_purge_handler;
701702
return;
702703
# endif
703704
default:
704-
r->write_event_handler = ngx_http_request_empty_handler;
705-
ngx_http_finalize_request(r, ngx_http_cache_purge_send_response(r));
705+
ngx_http_finalize_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR);
706706
}
707707
}
708708

0 commit comments

Comments
 (0)