Re: [PATCH] Cygwin: get_posix_access: avoid negative subscript

2019-08-27 Thread Ken Brown
On 8/27/2019 4:13 AM, Corinna Vinschen wrote: > On Aug 26 17:43, Ken Brown wrote: >> Don't refer to lacl[pos] unless we know that pos >= 0. > > I'm not sure this is entirely right. Moving the assignment to > class_perm/def_class_perm into the previous if makes sense, but the > bools

Re: [PATCH] Cygwin: get_posix_access: avoid negative subscript

2019-08-27 Thread Corinna Vinschen
Hi Ken, On Aug 26 17:43, Ken Brown wrote: > Don't refer to lacl[pos] unless we know that pos >= 0. I'm not sure this is entirely right. Moving the assignment to class_perm/def_class_perm into the previous if makes sense, but the bools has_class_perm and has_def_class_perm should be set no