Patch: Another attempt at VFS functions for setting and getting DOS attributes
Jeremy Allison
jra at samba.org
Fri Mar 25 23:11:26 UTC 2016
On Wed, Mar 23, 2016 at 10:02:05PM -0700, Jeremy Allison wrote:
> On Wed, Mar 23, 2016 at 09:54:15PM -0700, Richard Sharpe wrote:
> > Hi folks,
> >
> > I think this is getting closer.
> >
> > However, I still have to deal with one of Uri's concerns (the
> > fget_dos_attr function).
> >
> > I also used discard_const to squash an annoying error message. I plan
> > to submit another patch to constify some more struct smb_filenames ..
>
> > @@ -309,11 +309,12 @@ static bool get_ea_dos_attribute(connection_struct *conn,
> > case 1:
> > dosattr = dosattrib.info.info1.attrib;
> > if (!null_nttime(dosattrib.info.info1.create_time)) {
> > + struct smb_filename *fname = discard_const(smb_fname);
> > struct timespec create_time =
> > nt_time_to_unix_timespec(
> > dosattrib.info.info1.create_time);
>
> I don't think using discard_const is the right thing to
> do here.
>
> Let me take a closer look at this tomorrow.
FYI - I have a finished version of this under local
make test that I think is the right way to do this.
Didn't want you to waste more time on this - I'll
publish for review once it's passed make test.
More information about the samba-technical
mailing list