Incorrect delete in Dispatch JNI invokev()

[EJP286CRSKW]

In the error handling case at the end of the method we have:

buf = CreateErrorMsgFromInfo(hr, &excepInfo, nm);

and

dispIdAsName = new char[256];

and later:

if (buf) delete buf;
if (dispIdAsName) delete dispIdAsName;

The latter should be:

delete[] buf;
delete[] dispIdAsName;

to agree with how they were created, as new wchar_t[...] and new char[...] respectively.
The null tests are unnecessary.

[EJP286CRSKW]

Actually dispIdAsName could be declared as char dispIdAsName[256]; to avoid the delete [] altogether.

[EJP286CRSKW]

For clarity, the problems here are:

1. delete should be delete[] in both cases.
2. The null tests are redundant in both cases.

[freemansoft]

8e4472e