[ros-dev] [ros-diffs] [pschweitzer] 62779: [CDFS] Convert FCB pathname from simple buffer to unicode string. Please carefully review if I missed something

Pierre Schweitzer pierre at reactos.org
Sat Apr 19 11:04:36 UTC 2014


Thanks for your feedback. I fixed these.

On 19/04/2014 09:31, Thomas Faber wrote:
> On 2014-04-18 23:40, pschweitzer at svn.reactos.org wrote:
>> --- trunk/reactos/drivers/filesystems/cdfs/fcb.c	[iso-8859-1] (original)
>> +++ trunk/reactos/drivers/filesystems/cdfs/fcb.c	[iso-8859-1] Fri Apr 18 21:40:02 2014
>> @@ -129,7 +131,7 @@
>>  BOOLEAN
>>  CdfsFCBIsRoot(PFCB Fcb)
>>  {
>> -    return(wcscmp(Fcb->PathName, L"\\") == 0);
>> +    return (Fcb->PathName.Length = sizeof(WCHAR) && Fcb->PathName.Buffer[0] == L'\\');
>>  }
> ==
>
>
>
> Generally I'm seeing a lot of manual work where
> RtlInitEmptyUnicodeString and RtlCopyUnicodeString could be used.
>
> _______________________________________________
> Ros-dev mailing list
> Ros-dev at reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev


-- 
Pierre Schweitzer<pierre at reactos.org>
System Administrator
ReactOS Foundation


-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3968 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://www.reactos.org/pipermail/ros-dev/attachments/20140419/84709817/attachment.bin>


More information about the Ros-dev mailing list