Re: uninitialized variable access in read_builtin

[Chet Ramey]

On 6/22/23 12:36 PM, Grisha Levit wrote:
> `read' can hit its timeout before it gets a chance to save the current
> signal mask so sigprocmask can end up restoring an uninitialized
> prevset. (Also all the sigprocmask calls other than the one in the jmp
> target are protected by `#if defined (SIGCHLD)' so I guess this one
> should be too)
>
> Found by running the test suite on a build with clang's
> MemorySanitizer enabled.  There were only two other reports, both from
> quite recent additions, so I'll just mention them here:
> * anonopen doesn't set *fn if memfd_create is used so uw_anonclose
> frees an uninitialized pointer value later
> * convert_validarray_flags_to_arrayval_flags doesn't initialize avflags

Thanks. I appreciate you running the test suite.

--
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU    chet@case.edu    http://tiswww.cwru.edu/~chet/