Often Misused: Strings

マルチバイト文字列と Unicode 文字列の変換を実行する関数では、Buffer Overflow が発生しやすくなります。

Windows では、MultiByteToWideChar()、WideCharToMultiByte()、UnicodeToBytes()、および BytesToUnicode() といった関数で任意のマルチバイト文字列 (通常ANSI) と Unicode 文字列 (ワイド文字) を変換します。こうした関数へ渡すサイズ引数は、1 つの関数ではバイト数、他方では文字数と異なる単位で指定されるため、エラーが発生しやすくなります。マルチバイト文字列では各文字のバイト数が異なるため、文字列の長さはバイトの合計サイズを使うと簡単に指定できます。一方 Unicode では、文字は常に固定サイズであるため、文字列の長さは通常、含まれる文字数で決まります。サイズ引数を間違った単位で指定すると、Buffer Overflow の原因となる可能性があります。

例: 次の関数では、マルチバイト文字列として指定されたユーザー名およびユーザー情報を格納する構造体へのポインタを受け取り、構造体に指定ユーザーに関する情報を代入しています。Windows の Authentication ではユーザー名に Unicode が使用されるため、まずユーザー名引数をマルチバイト文字列から Unicode 文字列に変換しています。

void getUserInfo(char *username, struct _USER_INFO_2 info){
WCHAR unicodeUser[UNLEN+1];
MultiByteToWideChar(CP_ACP, 0, username, -1,
     unicodeUser, sizeof(unicodeUser));
NetUserGetInfo(NULL, unicodeUser, 2, (LPBYTE *)&info);
}

この関数は unicodeUser のサイズを、文字数でなく、誤ってバイト単位で渡しています。このため、MultiByteToWideChar() へのコールにより最大で (UNLEN+1)*sizeof(WCHAR) 個のワイド文字、または (UNLEN+1)*sizeof(WCHAR)*sizeof(WCHAR) バイトを unicodeUser 配列に書き込むことが可能ですが、この配列には (UNLEN+1)*sizeof(WCHAR) バイトしか割り当てられていません。username 文字列に含まれる文字数が UNLEN 字を超えている場合、MultiByteToWideChar() のコールにより、バッファ unicodeUser はオーバーフローします。