[PATCHv2 1/3] Add config option user-envvar

Ben Boeckel mathstuf at gmail.com
Mon Oct 29 15:53:41 CET 2012


On Mon, Oct 29, 2012 at 10:22:18 +0100, Valentin Haenel wrote:
> * Ben Boeckel <mathstuf at gmail.com> [2012-10-28]:
> > On Mon, Oct 22, 2012 at 08:29:16 GMT, Valentin Haenel wrote:
> > > @@ -166,6 +166,7 @@ struct cgit_query {
> > >  
> > >  struct cgit_config {
> > >  	char *agefile;
> > > +	char *user_envvar;
> > 
> > It should be free'd where the rest of these are free'd. I don't see that
> > here.
> > 
> > >  	char *cache_root;
> > >  	char *clone_prefix;
> > >  	char *clone_url;
> > > @@ -263,6 +264,7 @@ struct cgit_environment {
> > >  	char *script_name;
> > >  	char *server_name;
> > >  	char *server_port;
> > > +	char *remote_user;
> > 
> > Same here.
> 
> Forgive me if I am mistaken, but I don't see any of those free'd
> anywhere.

Hrm. That's...unfortunate. But likely a different patch if it's missing
altogether.

--Ben




More information about the CGit mailing list