Re: [PATCH] Geo: fix uninitialized memory access

2024-03-27 Thread Sergey Kandaurov
> On 21 Mar 2024, at 10:44, Piotr Sikora via nginx-devel > wrote: > > Hi Roman, > >> Thanks for the patch, looks valid, except we no longer need to explicitly >> initialize fields to zero. > > Right, I was going back-and-forth between which version I should send. > >> Also, I think we need m

Re: [PATCH] Geo: fix uninitialized memory access

2024-03-20 Thread Piotr Sikora via nginx-devel
Hi Roman, > Thanks for the patch, looks valid, except we no longer need to explicitly > initialize fields to zero. Right, I was going back-and-forth between which version I should send. > Also, I think we need more details about the > uninitialized memory access. See updated patch. LGTM, thank

Re: [PATCH] Geo: fix uninitialized memory access

2024-03-14 Thread Roman Arutyunyan
Hi Piotr, On Wed, Feb 28, 2024 at 01:21:40AM +, Piotr Sikora via nginx-devel wrote: > # HG changeset patch > # User Piotr Sikora > # Date 1708977621 0 > # Mon Feb 26 20:00:21 2024 + > # Branch patch005 > # Node ID fe6f8a72d42970df176ea53f4f0aea16947ba5b8 > # Parent 52936793ac076072c

[PATCH] Geo: fix uninitialized memory access

2024-02-27 Thread Piotr Sikora via nginx-devel
# HG changeset patch # User Piotr Sikora # Date 1708977621 0 # Mon Feb 26 20:00:21 2024 + # Branch patch005 # Node ID fe6f8a72d42970df176ea53f4f0aea16947ba5b8 # Parent 52936793ac076072c3544aa4e27f973d2f8fecda Geo: fix uninitialized memory access. Found with MemorySanitizer. Signed-off-