commit 96f210512651293c3505cc953eb44fa69e1b577f
author Vladimir Homutov <vl@nginx.com> Thu Aug 13 15:55:21 2015 +0300
committer Vladimir Homutov <vl@nginx.com> Thu Aug 13 15:55:21 2015 +0300
tree 27f6786258d73796e9d7c10fee5253b3ebebf721
parent 0a73df667032a753d97781623a0d6d5f4937f85f

Stream: fixed potential error log buffer overrun.

Found by Duan Jiong <djduanjiong@gmail.com>.

src/stream/ngx_stream_handler.c

diff --git a/src/stream/ngx_stream_handler.c b/src/stream/ngx_stream_handler.c
index 24afbcd..b3edb68 100644
--- a/src/stream/ngx_stream_handler.c
+++ b/src/stream/ngx_stream_handler.c
@@ -328,9 +328,11 @@
     p = ngx_snprintf(buf, len, ", client: %V, server: %V",
                      &s->connection->addr_text,
                      &s->connection->listening->addr_text);
+    len -= p - buf;
+    buf = p;
 
     if (s->log_handler) {
-        return s->log_handler(log, p, len);
+        p = s->log_handler(log, buf, len);
     }
 
     return p;