将uint16_t转换为wchar_t的安全方法

Question

试图清理一些代码，我想知道以下是否是将uint16_t转换为wchar_t的安全方法。

#if ! defined(MARKUP_SIZEOFWCHAR)
#if __SIZEOF_WCHAR_T__ == 4 || __WCHAR_MAX__ > 0x10000
#define MARKUP_SIZEOFWCHAR 4
#else
#define MARKUP_SIZEOFWCHAR 2
#endif

void FileReader::parseBuffer(char * buffer, int length)
{
  //start by looking for a vrsn
  //Header seek around for a vrns followed by 32 bit size descriptor
  //read 32 bits at a time
  int cursor = 0;
  char vrsn[5] = "vrsn";
  cursor = this->searchForMarker(cursor, length, vrsn, buffer);
  int32_t size = this->getObjectSizeForMarker(cursor, length, buffer);
  cursor = cursor + 7; //advance cursor past marker and size
  wchar_t *version = this->getObjectForSizeAndCursor(size, cursor, buffer);
  wcout << version;
  delete[] version; //this pointer is dest from getObjectForSizeAndCursor
}

--

wchar_t* FileReader::getObjectForSizeAndCursor(int32_t size, int cursor, char *buffer) {

  int wlen = size/2;
  uint32_t *dest = new uint32_t[wlen+1];
  unsigned char *ptr = (unsigned char *)(buffer + cursor);
  for(int i=0; i<wlen; i++) {
    #if MARKUP_SIZEOFWCHAR == 4 // sizeof(wchar_t) == 4
      char padding[2] = {'\0','\0'}; 
      dest[i] =  (padding[0] << 24) + (padding[1] << 16) + (ptr[0] << 8) + ptr[1];
    #else // sizeof(wchar_t) == 2
      dest[i] = (ptr[0] << 8) + ptr[1];
    #endif
      ptr += 2;
      cout << ptr;
  }
  return (wchar_t *)dest;
}

我使用填充的方式有任何范围问题吗？ 在调用函数中delete dest[]时，是否会泄漏填充？

Answer 1

区别

#if MARKUP_SIZEOFWCHAR == 4 // sizeof(wchar_t) == 4
  char padding[2] = {'\0','\0'}; 
  dest[i] =  (padding[0] << 24) + (padding[1] << 16) + (ptr[0] << 8) + ptr[1];
#else // sizeof(wchar_t) == 2
  dest[i] = (ptr[0] << 8) + ptr[1];
#endif

完全没有必要。 padding[i]为0，因此向左移动将其保持为0，将其相加则无效。

在每次循环迭代中，编译器可能会优化也可能不会优化两字节数组padding的分配，但是由于它是自动数组，因此不能以任何方式泄漏。

由于循环中使用的类型是无符号的，因此只需使用

dest[i] = (ptr[0] << 8) + ptr[1];

非常安全。 （字节序当然必须正确。）

对于

return (wchar_t *)dest;

您应该让dest的类型取决于wchar_t的大小，如果sizeof(wchar_t) == 2 （并且CHAR_BIT == 8 ），则应该为uint16_t* 。

Answer 2

您要尝试做的事不会起作用。 它以多种方式被破坏，但让我们关注一下演员表。

您的问题与您的代码不匹配。 您的代码正在使用uint32_t ，而您的问题询问的是uint16_t 。 但这没关系，因为两者都不起作用 。

如果需要使用wchar_t ，那么实际上应该使用 wchar_t 。 如果您的目标是获取char*两个连续字节并将其复制到wchar_t的前两个字节，则只需执行此操作。

这是代码的一个更好的版本，一个实际可行的版本 （在某种程度上可以从char*复制两个字节并假装它是wchar_t ）：

std::wstring FileReader::getObjectForSizeAndCursor(int32_t size, int cursor, char *buffer) {

  int wlen = size/2;
  std::wstring out(wlen);
  unsigned char *ptr = (unsigned char *)(buffer + cursor);
  for(int i=0; i<wlen; i++) {
    out[i] = (ptr[0] << 8) + ptr[1];
    ptr += 2;
    cout << ptr;
  }
  return out;
}

另外，由于我们使用的是诸如std::wstring类的适当RAII类，因此不会发生内存泄漏的情况。