Welcome! Log In Create A New Profile

Advanced

Re: [nginx] svn commit: r4812 - trunk/src/http/modules

magic.drums@gmail.com
August 12, 2012 01:36AM
If they were 10 commits not worry is not so much for the entire amount
already received but when there are more prefer to make a compiled and sent
on to the end of the day.

regards

On Sun, Aug 12, 2012 at 1:29 AM, thinkee365 <thinke365@gmail.com> wrote:

>
>
> why svn commit is send to maillist?
> it that necessary?
> if there are 10 commit to svn per day, then the subscriber of nginx maybe
> receive 10 mails:(
>
>
> On Wed, Aug 08, 2012 at 12:03:47PM +0000, vbart@nginx.com wrote:
> > Author: vbart
> > Date: 2012-08-08 12:03:46 +0000 (Wed, 08 Aug 2012)
> > New Revision: 4812
> > URL: http://trac.nginx.org/nginx/changeset/4812/nginx
> >
> > Log:
> > Added three missing checks for NULL after ngx_array_push() calls.
> >
> > Found by Coverity.
> >
> >
> > Modified:
> > trunk/src/http/modules/ngx_http_fastcgi_module.c
> > trunk/src/http/modules/ngx_http_limit_conn_module.c
> > trunk/src/http/modules/ngx_http_limit_req_module.c
> >
> > Modified: trunk/src/http/modules/ngx_http_fastcgi_module.c
> > ===================================================================
> > --- trunk/src/http/modules/ngx_http_fastcgi_module.c 2012-08-07
> 13:57:04 UTC (rev 4811)
> > +++ trunk/src/http/modules/ngx_http_fastcgi_module.c 2012-08-08
> 12:03:46 UTC (rev 4812)
> > @@ -1626,6 +1626,9 @@
> > }
> >
> > part = ngx_array_push(f->split_parts);
> > + if (part == NULL) {
> > + return NGX_ERROR;
> > + }
> >
> > part->start = part_start;
> > part->end = part_end;
> >
> > Modified: trunk/src/http/modules/ngx_http_limit_conn_module.c
> > ===================================================================
> > --- trunk/src/http/modules/ngx_http_limit_conn_module.c 2012-08-07
> 13:57:04 UTC (rev 4811)
> > +++ trunk/src/http/modules/ngx_http_limit_conn_module.c 2012-08-08
> 12:03:46 UTC (rev 4812)
> > @@ -721,6 +721,10 @@
> > }
> >
> > limit = ngx_array_push(&lccf->limits);
> > + if (limit == NULL) {
> > + return NGX_CONF_ERROR;
> > + }
> > +
> > limit->conn = n;
> > limit->shm_zone = shm_zone;
> >
> >
> > Modified: trunk/src/http/modules/ngx_http_limit_req_module.c
> > ===================================================================
> > --- trunk/src/http/modules/ngx_http_limit_req_module.c 2012-08-07
> 13:57:04 UTC (rev 4811)
> > +++ trunk/src/http/modules/ngx_http_limit_req_module.c 2012-08-08
> 12:03:46 UTC (rev 4812)
> > @@ -937,6 +937,9 @@
> > }
> >
> > limit = ngx_array_push(&lrcf->limits);
> > + if (limit == NULL) {
> > + return NGX_CONF_ERROR;
> > + }
> >
> > limit->shm_zone = shm_zone;
> > limit->burst = burst * 1000;
> >
> > _______________________________________________
> > nginx-devel mailing list
> > nginx-devel@nginx.org
> > http://mailman.nginx.org/mailman/listinfo/nginx-devel
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
>



--
Victor Pereira
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[nginx] svn commit: r4812 - trunk/src/http/modules

Anonymous User 1499 August 08, 2012 08:04AM

Re: [nginx] svn commit: r4812 - trunk/src/http/modules

thinkee365 559 August 12, 2012 01:30AM

Re: [nginx] svn commit: r4812 - trunk/src/http/modules

magic.drums@gmail.com 641 August 12, 2012 01:36AM

Re: [nginx] svn commit: r4812 - trunk/src/http/modules

thinkee365 548 August 12, 2012 01:42AM

Re: [nginx] svn commit: r4812 - trunk/src/http/modules

Sergey Budnevitch 586 August 12, 2012 04:20AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 79
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready