[BACK]Return to patch-plugins_lua_lua__plugin.c CVS log [TXT][DIR] Up to [cvs.NetBSD.org] / pkgsrc / www / py-uwsgi / patches

File: [cvs.NetBSD.org] / pkgsrc / www / py-uwsgi / patches / Attic / patch-plugins_lua_lua__plugin.c (download)

Revision 1.1, Mon Jan 11 22:57:00 2016 UTC (2 years, 7 months ago) by joerg
Branch: MAIN
CVS Tags: pkgsrc-2016Q3-base, pkgsrc-2016Q3, pkgsrc-2016Q2-base, pkgsrc-2016Q2, pkgsrc-2016Q1-base, pkgsrc-2016Q1

Avoid incrementing the same variable in the for body and at the end of
the loop. The code is correct, but triggers a useful clang 3.8 warning.

$NetBSD: patch-plugins_lua_lua__plugin.c,v 1.1 2016/01/11 22:57:00 joerg Exp $

--- plugins/lua/lua_plugin.c.orig	2016-01-11 16:11:52.000000000 +0000
+++ plugins/lua/lua_plugin.c
@@ -764,13 +764,12 @@ static int uwsgi_lua_request(struct wsgi
 	lua_newtable(L);
 	lua_pushstring(L, "");
 	lua_setfield(L, -2, "CONTENT_TYPE");
-	for(i=0;i<wsgi_req->var_cnt;i++) {
+	for(i=0;i<wsgi_req->var_cnt;i+=2) {
 		lua_pushlstring(L, (char *)wsgi_req->hvec[i+1].iov_base, wsgi_req->hvec[i+1].iov_len);
 		// transform it in a valid c string TODO this is ugly
 		ptrbuf = wsgi_req->hvec[i].iov_base+wsgi_req->hvec[i].iov_len;
 		*ptrbuf = 0;
 		lua_setfield(L, -2, (char *)wsgi_req->hvec[i].iov_base);
-		i++;
 	}