[ros-dev] [ros-diffs] [ion] 23175: - Major formatting cleanup and optimizations to NtQueryInformationProcess so that it does not burn the eyes of anyone that might attempt to read it.

Royce Mitchell III royce3 at ev1.net
Thu Jul 20 03:53:36 CEST 2006


ion at svn.reactos.org wrote:

>DstBuffer = (PWSTR)(DstPath + 1);
>  
>
that looks pretty eye-burning itself... perhaps I'm just seeing things but,

1) you're incrementing a pointer without checking if it's null ( I 
realize this might be safe if we can trust our caller ).

2) do we really want the address: DstPath + sizeof(PUNICODE_STRING)? I 
realize this too might work, but do we want to assume that 
sizeof(PUNICODE_STRING) == 2*sizeof(USHORT) ???



More information about the Ros-dev mailing list