Bug 4054

Summary: Надо добавить в winecfg вкладку для управления загрузкой драйверов ключей
Product: WINE@Etersoft Reporter: Александр Морозов <amorozov>
Component: Ключи защиты ; Системы защиты ; Файл лицензииAssignee: Илья Шпигорь <shpigor>
Status: CLOSED FIXED QA Contact: Денис Баранов <baraka>
Severity: minor    
Priority: P4 CC: lav
Version: 1.1   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Заявки RT: Связано с:
Дата напоминания:
Bug Depends on:    
Bug Blocks: 3589, 3946, 6916    
Deadline: 2009-07-31   
Attachments: Вкладка Dongles (Ключи защиты)
Лог падения winecfg

Description Александр Морозов 2009-06-26 13:04:04 MSD
Created attachment 1233 [details]
Вкладка Dongles (Ключи защиты)

Для выбранного в списке драйвера должна быть возможность указать, загружать его или нет (флажок "Load a driver"). Также для него должно определяться, загружен драйвер или нет (должно отображаться либо "Driver is loaded", либо "Driver is not loaded").

Для проверки, загружен ли драйвер, надо вызывать
static int is_loaded(dongle_t dongle);
Если не загружен, функция будет возвращать 0. При установке флажка "Load a driver" надо вызывать
static void set_start(dongle_t dongle);

enum dongle_t
{
    guardant,
    hasp,
    katran_lpt,
    katran_usb,
    sentinel,
    smartkey3
};
Comment 1 Александр Морозов 2009-06-26 13:35:49 MSD
Поправочка:
static void set_start(dongle_t dongle, int start);
start должно быть равно 1, когда надо загружать драйвер, и 0, когда не надо
Comment 2 Александр Морозов 2009-06-29 20:02:50 MSD
Created attachment 1235 [details]
Лог падения winecfg

Посмотрел патч. Надо увеличить NUM_PROPERTY_PAGES на 1, а то вкладка "О программе" пропала.

Мне кажется, лучше заменить "Загруженные ключи защиты" на "Загрузка драйверов ключей", так как загружаются не ключи, а их драйверы. "Loading dongles" можно заменить на "Loading dongles drivers".

"Загружен/Не загружен" надо заменить на "Загружать/Не загружать", так как в этом столбце выводится информация о том, не запрещена ли загрузка драйвера, а не о том, загружен он сейчас или нет. Вкладка не занимается загрузкой/выгрузкой драйверов прямо сейчас, она меняет настройки загрузки драйверов при первом запуске wine. "Loaded/Unloaded" надо заменить на "Load/Unload", "Загрузить/Выгрузить" на "Загружать/Не загружать", "Unload" надо заменить на "Do not load".

А ещё winecfg почему-то падает при выборе новой вкладки.
Comment 3 Денис Баранов 2009-07-16 21:45:30 MSD
Принято.
WINE@Etersoft eter27/eter19
Comment 4 Vitaly Lipatov 2009-07-31 11:17:03 MSD
Всё равно грузится hardlock в чистом 1.0.11:
trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\parport.sys" at 0x8f460000: builtin   
trace:loaddll:load_builtin_dll Loaded L"C:\\windows\\system32\\hal.dll" at 0x8f640000: builtin       
trace:loaddll:load_native_dll Loaded L"C:\\windows\\system32\\drivers\\hardlock.sys" at 0x81550000: native
preloader: Warning: failed to reserve range 00000000-00010000                                             
trace:loaddll:load_builtin_dll Loaded L"KERNEL32.dll" at 0x8fe70000: builtin   

В итоге загружается
14248 ?        Sl     0:00 C:\windows\system32\winedevice.exe Hardlock
14256 ?        Sl     0:00 C:\windows\system32\winedevice.exe
14262 ?        Sl     0:00 C:\windows\system32\winedevice.exe MountMgr
14272 ?        Sl     0:00 C:\windows\system32\winedevice.exe Parport
14282 ?        Sl     0:00 C:\windows\system32\winedevice.exe Usbhub
Comment 5 Александр Морозов 2009-07-31 12:24:26 MSD
Поправил. Файл реестра для hardlock.sys находится в закрытой части, поэтому я про него забыл.
Comment 6 Денис Баранов 2009-08-01 21:15:56 MSD
Нужно чтобы после нажатия на кнопку "Применить" выводилось сообщение о перезапуске wineserver или предложение перезагрузить его самостоятельно чуть позже, т.к. получается не логичность присутствие кнопки "Применить", и для кнопки "Ок" такое тоже нужно.
Comment 7 Денис Баранов 2009-08-01 21:39:26 MSD
(In reply to comment #6)
> Нужно чтобы после нажатия на кнопку
> "Применить" выводилось сообщение о
> перезапуске wineserver или предложение
> перезагрузить его самостоятельно чуть
> позже, т.к. получается не логичность
> присутствие кнопки "Применить", и для
> кнопки "Ок" такое тоже нужно.
> 
Создал отдельную багу:
http://bugs.etersoft.ru/show_bug.cgi?id=4171