[tHeWiZaRdOfDoS]

Quote

bool CemuleApp::IsEd2kLinkInClipboard(LPCSTR pszLinkType, int iLinkTypeLen)
{
bool bFoundLink = false;
if (IsClipboardFormatAvailable(CF_TEXT))
{
  if (OpenClipboard(NULL))
  {
  HGLOBAL hText = GetClipboardData(CF_TEXT);
  if (hText != NULL)
  {
    // Use the ANSI string
    LPCSTR pszText = (LPCSTR)GlobalLock(hText);
    if (pszText != NULL)
    {
    while (*pszText == ' ' || *pszText == '\t' || *pszText == '\r' || *pszText == '\n')
      pszText++;
    bFoundLink = (strncmp(pszText, pszLinkType, iLinkTypeLen) == 0);
    //GlobalUnlock(hText);
    }
    GlobalUnlock(hText); //WiZaRd
  }
...

It should never happen, though, it's better to unlock correctly

[SlugFiller]

I don't think you understand what "GlobalLock" does.
It's less like "CSingleLock::Lock", and more like "malloc".
"GlobalUnlock" is therefore like "free".

So this code is sort of like:

    LPCSTR pszText = (LPCSTR)malloc(...);
    if (pszText != NULL)
    {
    ...
    free(pszText);
    }

Your "fix" is hence:

    LPCSTR pszText = (LPCSTR)malloc(...);
    if (pszText != NULL)
    {
    ...
    //free(pszText);
    }
    free(pszText);

I hope the issue is visible.

[tHeWiZaRdOfDoS]

Oah *narf* I switch those 2:

Quote

bool CemuleApp::IsEd2kLinkInClipboard(LPCSTR pszLinkType, int iLinkTypeLen)
{
...
...
    // Use the ANSI string
    LPCSTR pszText = (LPCSTR)GlobalLock(hText);
    if (pszText != NULL)