Re: [ros-dev] [ros-diffs] [cgutman] 41783: - Fix return value of tdiGetMibForIfEntity so it doesn't return a failure status on success

2009-07-06 Thread Ged
This looks hackish It also breaks the log as you're potentially returning something different from what the log states -Original Message- From: ros-diffs-boun...@reactos.org [mailto:ros-diffs-boun...@reactos.org] On Behalf Of cgut...@svn.reactos.org Sent: 06 July 2009 08:54 To:

Re: [ros-dev] [ros-diffs] [cgutman] 41783: - Fix return value of tdiGetMibForIfEntity so it doesn't return a failure status on success

2009-07-06 Thread Cameron Gutman
It's not a hack. We know we have succeded if we reach that code because we check for failure earlier. The problem is that status was the return value of DeviceIoControl which returns nonzero for success but zero is success in ntstatus. On Jul 6, 2009, at 4:44 AM, Ged gedmur...@gmail.com

Re: [ros-dev] [ros-diffs] [cgutman] 41783: - Fix return value of tdiGetMibForIfEntity so it doesn't return a failure status on success

2009-07-06 Thread Ged
Yeah, I should have stated this more clearly. My point was DeviceIoControl returns TRUE, you then print this to the log as an NTSTATUS value which now holds the value '1' which in NTSTATUS terms is STATUS_WAIT_0 + 1 After this you then return STATUS_SUCCESS which is 0. As I said, it's

Re: [ros-dev] [ros-diffs] [cgutman] 41783: - Fix return value of tdiGetMibForIfEntity so it doesn't return a failure status on success

2009-07-06 Thread Cameron Gutman
Yeah, I need to fix those. On Jul 6, 2009, at 4:39 PM, Timo Kreuzer timo.kreu...@web.de wrote: The hack is storing a BOOL value in an NTSTATUS variable. Cameron Gutman schrieb: It's not a hack. We know we have succeded if we reach that code because we check for failure earlier. The