dpwsockx is not used by any application, only by us in dplayx, so it's
pretty safe to assume we won't pass any invalid parameter, as all the
parameters passed by applications are already checked for validity in
dplayx.

On Tue, Sep 1, 2009 at 11:00 AM, Henri Verbeet<hverb...@gmail.com> wrote:
> 2009/9/1 Ismael Barros <razielm...@gmail.com>:
>> ---
>>  dlls/dpwsockx/Makefile.in     |    2 +-
>>  dlls/dpwsockx/dpwsockx_main.c |  164 
>> ++++++++++++++++++++++++++++++++++++++++-
>>  2 files changed, 163 insertions(+), 3 deletions(-)
>>
> This patch seems to assume the application will never pass NULL for
> "data", is that safe to assume? If applications aren't allowed to pass
> NULL it's ok to crash, although it might make your life simpler if it
> isn't inside a TRACE/FIXME.
>
>
>


Reply via email to