Patch: Another attempt at VFS functions for setting and getting DOS attributes

Jeremy Allison jra at samba.org
Thu Mar 24 05:02:05 UTC 2016


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.

Jeremy.



More information about the samba-technical mailing list