Juan Lang wrote: > Hi Rob, > > - sd.sacl_len = (present ? sacl->AclSize : 0); > + sd.sacl_len = ((present && dacl) ? sacl->AclSize : 0); > > Shouldn't that be ((present && sacl) ? sacl->AclSize : 0); ? Good spot. It wouldn't be the first time I've mixed up dacls and sacls... Thanks, -- Rob Shearman