Re: [ros-dev] [ros-diffs] [hbelusca] 74493: [NTOS]: Capture the counted BaseDllName unicode string into a local NULL-terminated buffer before calling wcsrchr on it (actually I think it would be better

2017-05-08 Thread Thomas Faber
There are 12 ExAllocatePoolWithTag calls in that file, and 2 ExAllocatePool calls, one of which you added. The consistency argument is really not on your side. But my point is that when adding new code, you should please follow the "new" guideline (you know, the one that's going on 20 years old).

Re: [ros-dev] [ros-diffs] [hbelusca] 74493: [NTOS]: Capture the counted BaseDllName unicode string into a local NULL-terminated buffer before calling wcsrchr on it (actually I think it would be better

2017-05-07 Thread Hermès BÉLUSCA-MAÏTO
Hi! The assert is fixed (r74495). For ExAllocatePool, it would be better to change them all at once in this file. Hermès -Message d'origine- De : Ros-dev [mailto:ros-dev-boun...@reactos.org] De la part de Thomas Faber Envoyé : dimanche 7 mai 2017 20:49 À : ros-dev@reactos.org Objet :

Re: [ros-dev] [ros-diffs] [hbelusca] 74493: [NTOS]: Capture the counted BaseDllName unicode string into a local NULL-terminated buffer before calling wcsrchr on it (actually I think it would be better

2017-05-07 Thread Thomas Faber
On 2017-05-07 20:15, hbelu...@svn.reactos.org wrote: --- trunk/reactos/ntoskrnl/io/iomgr/driver.c[iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/driver.c[iso-8859-1] Sun May 7 18:15:12 2017 @@ -884,7 +884,7 @@ PDEVICE_NODE DeviceNode; PDRIVER_OBJECT DriverObject;