advpack: Use the FAILED and SUCCEEDED macros instead of comparing to S_OK or zero.

[Mike McCormack]

James Hawkins wrote:

> Would you accept the patch if I put all results into an hr and then
> called FAILED(hr) instead?

Yeah, I think that's a better idea.

Mike