[Win32API] OpenClipboard 関数を複数回呼ぶと落ちてしまう

フォーラム(掲示板)ルール
フォーラム(掲示板)ルールはこちら  ※コードを貼り付ける場合は [code][/code] で囲って下さい。詳しくはこちら

トピックに返信する


答えを正確にご入力ください。答えられるかどうかでスパムボットか否かを判定します。

BBCode: ON
[img]: ON
[flash]: OFF
[url]: ON
スマイリー: OFF

トピックのレビュー
   

展開ビュー トピックのレビュー: [Win32API] OpenClipboard 関数を複数回呼ぶと落ちてしまう

Re: [Win32API] OpenClipboard 関数を複数回呼ぶと落ちてしまう

#3

by Bull » 3年前

2カ所ほど修正すれば正常に動作するようです。
wcscpy_s() 関数の第二引数(numberOfElements )は文字数であってバイト数ではないです。
ptr[num_bytes - 1] = L'\0'; は必要ない上に確保したメモリ以外の領域に書込んでいます。

コード:

if (ptr) {
	wcscpy_s(ptr, num_wchars, wtext);
	GlobalUnlock(hMem);
}
とすると、当方の環境ではエラーがなくなりました。

Re: [Win32API] OpenClipboard 関数を複数回呼ぶと落ちてしまう

#2

by usao » 3年前

関係あるかどうかわかりませんが,
EmptyClipboardとかSetClipboardDataが成功しているかを戻り値でチェックしてみてはどうでしょうか.

msdnでOpenClipboardやEmptyClipboardの説明を見ると,そのような呼び方をするとSetClipboardDataが失敗する,と書かれているように見えます.

[Win32API] OpenClipboard 関数を複数回呼ぶと落ちてしまう

#1

by kanov » 3年前

Visual Studio 2017 で C++ を使っているのですが、クリップボードに文字列をコピーしようとしたら、
1度目は成功しますが2度目で必ず失敗してしまいます。

2度目にOpenClipboard (Win32API) を実行しようとしたところで(関数が戻ってくる前に)
"OpenClipboardに失敗しました(HRESULTからの例外: 0x800401D0 (CLIPBRD_E_CANT_OPEN))"
となってしまうのですが、どこが悪いのでしょうか???

コード:

#include <Windows.h>

bool SetClipboardTextW(const wchar_t *wtext) {
	HGLOBAL hMem = NULL;
	{
		size_t num_wchars = wcslen(wtext) + 1; // 終端文字含む
		size_t num_bytes = sizeof(wchar_t) * num_wchars;
		hMem = GlobalAlloc(GHND|GMEM_SHARE, num_bytes);
		wchar_t *ptr = (wchar_t *)GlobalLock(hMem);
		if (ptr) {
			wcscpy_s(ptr, num_bytes, wtext);
			ptr[num_bytes-1] = L'\0'; // wcscpy_sは終端文字も書き込むはずだが、念のため
			GlobalUnlock(hMem);
		}
	}
	if (OpenClipboard(NULL)) { // <--- ここで落ちる
		EmptyClipboard();
		SetClipboardData(CF_UNICODETEXT, hMem);
		CloseClipboard();
		return true; // GlobalFreeしてはいけない
	} else {
		GlobalFree(hMem);
		return false;
	}
}

int WINAPI WinMain(HINSTANCE hCurrInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow) {
	SetClipboardTextW(L"HELLO WORLD");
	SetClipboardTextW(L"HELLO WORLD"); // <--- これが成功しない
	return 0;
}


ページトップ