[ofw] RE: [PATCH] UAL: Fix NULL dereference when polling a CEP

Fab Tillier ftillier at windows.microsoft.com
Thu Apr 9 11:11:34 PDT 2009


>> I changed the name of the 'context' field in the IOCTL to catch all
>> references to it at compile time.
>
> If this field is exposed in a header that can be included by a
> userspace app, we shouldn't change the name.

The field no longer contains any data that would be useful to an app that issues its own IOCTLs, so changing the name helps to catch the erroneous use of the field, rather than letting things go on to blow up with a NULL pointer dereference.

Is issuing IOCTLs by non-driver components something that is/should be supported (let alone done?)

-Fab




More information about the ofw mailing list