CODEWITHSUNDEEP
c++winapiconsolewchar-t
rsk82
rsk82

Reputation: 29397

wcslen quits silently when string returned by the ReadConsoleOutputCharacterW has some particular length

compiler : http://sourceforge.net/projects/mingwbuilds/files/

#include <iostream>
#include <string.h>
#include <windows.h>
using namespace std;

  const wchar_t* readConsole(int chars_to_read) {
    wchar_t* wcharFromConsole = new wchar_t[chars_to_read+1];
    COORD pos = {0,0};
    DWORD dwChars;
    if (!ReadConsoleOutputCharacterW(
      GetStdHandle(STD_OUTPUT_HANDLE),
      wcharFromConsole,  // Buffer where store symbols
      chars_to_read,     // number of chars to read
      pos,    // Read from row=8, column=6
      &dwChars // How many symbols stored
    ))
    {
      printf("ReadConsoleOutputCharacterW failed %d\n", GetLastError());
      abort();
    }
    wcharFromConsole [dwChars] = L'\0'; // Terminate, so string functions can be used
    wstring ws = wcharFromConsole;
    return ws.c_str();
  }

int main() {
  for (int i = 1; i<=0x3000; i++) {
    printf("wcslen: %X \n",wcslen(readConsole(i)));
  }
  system("pause");
}

This loop ends at 0x1FF1 and pause is not called. Removing wstring seems to do away with this problem. But I need it here for functions like trimming white-space etc.. it is not much relevant here, but why invoking wstring causes that issue anyway ? There is no error message the program simply quits.

Updated code, now loop quits at 0x2BBF

#include <iostream>
#include <string.h>
#include <windows.h>
using namespace std;

  const wchar_t* readConsole(int chars_to_read) {
    wchar_t* wcharFromConsole = new wchar_t[chars_to_read+1];
    COORD pos = {0,0};
    DWORD dwChars;
    if (!ReadConsoleOutputCharacterW(
      GetStdHandle(STD_OUTPUT_HANDLE),
      wcharFromConsole,  // Buffer where store symbols
      chars_to_read,     // number of chars to read
      pos,    // Read from row=8, column=6
      &dwChars // How many symbols stored
    ))
    {
      printf("ReadConsoleOutputCharacterW failed %d\n", GetLastError());
      abort();
    }
    wcharFromConsole [dwChars] = L'\0'; // Terminate, so string functions can be used
    wstring ws = wcharFromConsole;
    delete [] wcharFromConsole;
    const wchar_t* wc = ws.c_str();
    return wc;
  }

int main() {
  for (int i = 1; i<=0x3000; i++) {
    printf("wcslen: %X \n",wcslen(readConsole(i)));
  }
  system("pause");
}

Upvotes: 0

Views: 313

Answers (1)

Christian Stieber
Christian Stieber

Reputation: 12496

Ouch.

wstring ws = wcharFromConsole;
return ws.c_str();

Basically, you are returning a dead pointer here. The string will be destroyed on the return, so the pointer arriving at the caller will be invalid.

EDIT: you're also leaking memory, since the "new" is never deleted. But that doesn't generally cause visible problems, just increasing memory use of the program.

Upvotes: 2

Related Questions