[trafficserver] branch master updated: TS-5053 const char **argv passed to TSPluginInit is not null terminated

[sh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

shinrich pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  3b9fec5   TS-5053 const char **argv passed to TSPluginInit is not null terminated
3b9fec5 is described below

commit 3b9fec52ab1cec8795c3f80e02619118901cf6b7
Author: Daniel Xu <dl...@yahoo.com>
AuthorDate: Mon Dec 12 12:38:05 2016 -0600

    TS-5053 const char **argv passed to TSPluginInit is not null terminated
    
    Typically **argv is null terminated in other systems. There's no
    good reason for ATS to be any different.
---
 proxy/Plugin.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/proxy/Plugin.cc b/proxy/Plugin.cc
index b14e6ae..5ae13c9 100644
--- a/proxy/Plugin.cc
+++ b/proxy/Plugin.cc
@@ -300,6 +300,7 @@ plugin_init(bool validateOnly)
         argv[i] = vars[i];
       }
     }
+    argv[argc] = nullptr;
 
     retVal = plugin_load(argc, argv, validateOnly);
 

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].