[nginx] SPDY: moved a variable initialization near to its check.

Valentin Bartenev vbart at nginx.com
Thu Apr 10 13:49:50 UTC 2014


details:   http://hg.nginx.org/nginx/rev/363395795384
branches:  
changeset: 5650:363395795384
user:      Valentin Bartenev <vbart at nginx.com>
date:      Wed Apr 09 18:15:32 2014 +0400
description:
SPDY: moved a variable initialization near to its check.

This should prevent attempts of using pointer before it was checked, since
all modern compilers are able to spot access to uninitialized variable.

No functional changes.

diffstat:

 src/http/ngx_http_spdy.c |  4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diffs (21 lines):

diff -r e1dcb983d6b3 -r 363395795384 src/http/ngx_http_spdy.c
--- a/src/http/ngx_http_spdy.c	Tue Apr 08 20:12:30 2014 +0400
+++ b/src/http/ngx_http_spdy.c	Wed Apr 09 18:15:32 2014 +0400
@@ -1412,8 +1412,6 @@ ngx_http_spdy_state_data(ngx_http_spdy_c
 {
     ngx_http_spdy_stream_t  *stream;
 
-    stream = sc->stream;
-
     ngx_log_debug0(NGX_LOG_DEBUG_HTTP, sc->connection->log, 0,
                    "spdy DATA frame");
 
@@ -1441,6 +1439,8 @@ ngx_http_spdy_state_data(ngx_http_spdy_c
         sc->recv_window = NGX_SPDY_MAX_WINDOW;
     }
 
+    stream = sc->stream;
+
     if (stream == NULL) {
         return ngx_http_spdy_state_skip(sc, pos, end);
     }



More information about the nginx-devel mailing list