What's wrong with this piece of code from source3/modules/vfs_acl_xattr.c around like 126 or so?

[Jeremy Allison]

On Mon, May 14, 2012 at 02:28:59PM -0700, Richard Sharpe wrote:
>         if (ret) {
>                 errno = saved_errno;
>                 DEBUG(5, ("store_acl_blob_fsp: setting attr failed for file %s"
>                         "with error %s\n",
>                         fsp_str_dbg(fsp),
>                         strerror(errno) ));
>                 return map_nt_error_from_unix(errno);
>         }
> 
> Hint. It has to do with saved_errno.

Oh, you mean the errno = saved_errno should
be *after* the DEBUG statement ?

Just checking - not sure exactly what I missed here.

Jeremy.