View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001926 | Plugins | EMenu | public | 2011-11-27 18:54 | 2011-11-28 14:27 |
Reporter | Maximus | Assigned To | DrKnS | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 3.0 | ||||
Fixed in Version | 3.0 | ||||
Summary | 0001926: Падение при выполнении "rclk_cmd:Paste" | ||||
Description | Падает на вызове pContextMenu->GetCommandString(nID, nType, NULL, reinterpret_cast<LPSTR>(szwAddInfo), ARRAYSIZE(szwAddInfo)); Допускаю, что глюк из-за некорректно зарегистрированного extension, но нехорошо это, что фар схлопывается. | ||||
Additional Information | в буфере лежит файл T:\Utils\Lans\GCC-4.6.1.x64\COPYING.MinGW-w64-runtime.txt а текущая папка (пустая) F:\Temp\3 Происходит сие на пункте меню "Nero BackItUp". | ||||
Tags | No tags attached. | ||||
|
111127.diff (766 bytes)
Index: Plugin.cpp =================================================================== --- Plugin.cpp (revision 6893) +++ Plugin.cpp (working copy) @@ -968,9 +968,16 @@ return false; } WCHAR szwAddInfo[200]=L"\0"; - if (FAILED(pContextMenu->GetCommandString(nID, nType, NULL, reinterpret_cast<LPSTR>(szwAddInfo), ARRAYSIZE(szwAddInfo)))) + HRESULT hr; + __try { + hr = pContextMenu->GetCommandString(nID, nType, NULL, reinterpret_cast<LPSTR>(szwAddInfo), ARRAYSIZE(szwAddInfo)); + } __except(EXCEPTION_EXECUTE_HANDLER) { + hr = E_UNEXPECTED; + } + if (FAILED(hr)) { - return false; + if (FAILED(hr)) + return false; } *pstr=szwAddInfo; if (AS_HELPTEXT==enAdditionalString) pstr->RemoveTrailing(L'.'); |
|
111127fix.diff (631 bytes)
Index: Plugin.cpp =================================================================== --- Plugin.cpp (revision 6893) +++ Plugin.cpp (working copy) @@ -968,7 +968,13 @@ return false; } WCHAR szwAddInfo[200]=L"\0"; - if (FAILED(pContextMenu->GetCommandString(nID, nType, NULL, reinterpret_cast<LPSTR>(szwAddInfo), ARRAYSIZE(szwAddInfo)))) + HRESULT hr; + __try { + hr = pContextMenu->GetCommandString(nID, nType, NULL, reinterpret_cast<LPSTR>(szwAddInfo), ARRAYSIZE(szwAddInfo)); + } __except(EXCEPTION_EXECUTE_HANDLER) { + hr = E_UNEXPECTED; + } + if (FAILED(hr)) { return false; } |
|
в 111127.diff проверка лишняя попала |
|
build 45. |
|
emenu не собирается |
|
fixed. |
|
vcxproj тоже надо поправить |
|
угу. |
Date Modified | Username | Field | Change |
---|---|---|---|
2011-11-27 18:54 | Maximus | New Issue | |
2011-11-27 18:54 | Maximus | File Added: 111127.diff | |
2011-11-27 18:55 | Maximus | File Added: 111127fix.diff | |
2011-11-27 18:56 | Maximus | Note Added: 0007994 | |
2011-11-28 11:37 | DrKnS | Note Added: 0007995 | |
2011-11-28 11:37 | DrKnS | Status | new => closed |
2011-11-28 11:37 | DrKnS | Assigned To | => DrKnS |
2011-11-28 11:37 | DrKnS | Resolution | open => fixed |
2011-11-28 11:37 | DrKnS | Fixed in Version | => 3.0 |
2011-11-28 13:20 | 2useven10 | Note Added: 0007996 | |
2011-11-28 13:20 | 2useven10 | Status | closed => feedback |
2011-11-28 13:20 | 2useven10 | Resolution | fixed => reopened |
2011-11-28 13:36 | DrKnS | Note Added: 0007997 | |
2011-11-28 13:36 | DrKnS | Status | feedback => closed |
2011-11-28 13:36 | DrKnS | Resolution | reopened => fixed |
2011-11-28 14:12 | 2useven10 | Note Added: 0007998 | |
2011-11-28 14:27 | DrKnS | Note Added: 0007999 |