This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: More passwd/group patches


At 08:02 PM 11/24/2002 +0100, you wrote:
>+  if (aclbufp) {
>+    if (EqualSid (owner_sid, group_sid))
>+      lacl[0].a_perm = lacl[1].a_perm;
>+    aclsort (pos, 0, aclbufp);
>+    if (pos > nentries)
>+      pos = nentries;
>     memcpy (aclbufp, lacl, pos * sizeof (__aclent16_t));
>-  aclsort (pos, 0, aclbufp);
>+  }
>
>Do you see the problem?

Oops, yes. I have aclsort (pos, 0, ACLU);
There is another strange thing in there that I should have mentioned:
we are returning success even if the user gives us an ACLU that's 
tohasmall. Is that how Sun does it? 
I thought that if we were doing that, we should return the first
Acts after sorting them. That's why I moved the sort up. 

itsere


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]