-
Notifications
You must be signed in to change notification settings - Fork 174
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Crash when viewing a file from an archive while a search is in progress #2526
Comments
@spnethw Покажите, пожалуйста, записи в ~/.config/far2l/crash.log, если есть |
crash.log почти никогда не успевает создаться, попробовал подцепиться через gdb:
|
@spnethw у меня воспроизвелось. Похоже, идет борьба за строчку с названием временного каталога. Поправил наугад, вроде не падает, попробуйте, пожалуйста. |
К сожалению, не помогает. 😞 Продолжает падать...
|
@spnethw перенес блокировку повыше, попробуйте. |
А вот теперь, вроде бы, не воспроизводится! |
эмм, немного не понимаю что оно фиксит, потому что гонки не вижу. А вот вижу использование GetTempFileName (в FarMkTempEx) с буфером непонятного размера, в то время как ей требуется MAX_PATH, потому корректный фикс имхо вместо |
Заменил на Не-а, не помогает, падает. |
@elfmz вот такой отчет для варианта MAX_PATH, посмотрите два потока, которые работали
|
а, UseFarCommand, может изза него. Тогда по идее достаточно только один лок передвинуть, вот так:
..если только такое изменение сделать, перестанет падать? |
да, с таким патчем на моих примерах вроде бы перестало падать! |
и у меня не падает, переделал PR |
far2l рандомно падает в следующей ситуации:
от типа архивов, вроде бы, не зависит, падало на zip, rar, tar.gz
воспроизводится не каждый раз, иногда с десятой-двадцатой попытки.
The text was updated successfully, but these errors were encountered: