Welcome! Log In Create A New Profile

Advanced

Re: Re: fix accidental corrdump

Maxim Dounin
September 28, 2022 11:58AM
Hello!

On Wed, Sep 28, 2022 at 10:56:15AM +0800, zjd wrote:

> my module is such:
> The pool is created with ngx_create_pool in
> module_init_process; I get momery from the pool in the start of
> each request, and then use ngx_reset_pool in the end of each
> request.
> The ngx_reset_pool take pool->large each alloc pointer return
> to pool, and each alloc pointer is wild pointer.
> And When another get momery from the pool in next request or
> next next request..., I maybe get wild pointer address and
> access inaccessible addresses, and then coredump.
>
> maybe the describe is simple in last mail.

Ok, so from your description you are getting segfaults, and you
don't know why.

Note that the ngx_reset_pool() function clears pool->large, and
also frees all the ngx_pool_large_t structures (since it resets
all pool blocks). That is, l->alloc you are clearing in your
patch is not expected to be used anywhere. If clearing it helps,
this suggests that there is a bug in your module which results in
this freed memory to be used somehow.

While clearing l->alloc might appear to help, most likely it is
just hiding a bug in your module. Correct solution would be to
find the bug in your module and fix it.

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx mailing list -- nginx@nginx.org
To unsubscribe send an email to nginx-leave@nginx.org
Subject Author Posted

fix accidental corrdump

zjd September 27, 2022 03:26AM

Re: fix accidental corrdump

Maxim Dounin September 27, 2022 04:54PM

Re:Re: fix accidental corrdump

zjd September 27, 2022 10:58PM

Re: Re: fix accidental corrdump

Maxim Dounin September 28, 2022 11:58AM

Re:Re: Re: fix accidental corrdump

zjd September 29, 2022 04:32AM

Re: Re: Re: fix accidental corrdump

Maxim Dounin September 29, 2022 03:54PM

Re: Re: Re: fix accidental corrdump

Frank Swasey September 29, 2022 04:38PM

Re: Re: Re: fix accidental corrdump

Maxim Dounin September 29, 2022 07:22PM

Re:Re: Re: Re: fix accidental corrdump

zjd September 30, 2022 12:10AM

Re: Re: Re: Re: fix accidental corrdump

Maxim Dounin September 30, 2022 08:08AM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 177
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