Patch: Certain error conditions are handled incorrectly in COM methods

Hey.

In TSVN certain error conditions in COM methods are handled incorrectly:
1. Sometimes no checks for null pointers are performed and those pointers are later dereferenced.
2. Sometimes E_INVALIDARG is returned when a pointer to an "out" parameter is null. E_POINTER should be returned instead.
3. Sometimes S_FALSE is returned for incorrect arguments. S_FALSE evaluates "FAILED()" macro to "false" and is a way to indicate that the call has completed successfully but either the answer to the query is "no, thank you" or the query has obtained less results than requested. Such misuse is an example of error hiding.
4. Sometimes an object is created with "new" and IUnknown::QueryInterface() is called in it without a check for whether that call succeeded and that can cause a memory leak.

A patch is proposed.

Dmitry.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2395816

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].