View Issue Details

IDProjectCategoryView StatusLast Update
0001684Far ManagerPanelpublic2011-12-23 08:53
ReporterMaximus Assigned Tovskirdin  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Platformx86OSWindowsOS Version7
Product Version2.0 
Fixed in Version2.0 
Summary0001684: Падение FileList:GetCurName во время чтения сетевой папки
DescriptionПадает из-за того, что из CheckForEscSilent вызывается ProcessSynchroEvent. А плагин зовет ACTL_GETWINDOWINFO.
Фар еще не закончил чтение каталога и CurFile больше чем FileCount.
Что приводит к падению.

Наверное нужно запретить синхро из CheckForEscSilent?
Да и assert-ов в filelist.cpp повтыкать не помешало бы, чтобы видеть проблему сразу, а не тогда, когда случайно на строковой функции свалится
TagsNo tags attached.
Build1804

Activities

Maximus

2011-01-20 21:19

reporter  

110120.diff.7z (2,011 bytes)

Maximus

2011-01-20 21:19

reporter   bugnote:0006227

как-то так, например?

vskirdin

2011-01-28 06:23

administrator   bugnote:0006264

а зачем лишнее условие?

if (PrevMouseButtonState!=MouseButtonState) PrevMouseButtonState=MouseButtonState;

Maximus

2011-01-28 19:59

reporter  

110128.diff.7z (1,919 bytes)

Maximus

2011-01-28 20:00

reporter   bugnote:0006267

отлаживался, мешалось - брекпойнт по значению всплывал постоянно

убрал условие, поправил один ассерт в конце

vskirdin

2011-01-29 06:02

administrator   bugnote:0006268

в FileList::GetSelCount() спорный момент на счет проверки в ассерте "!FileCount".
Может имеет смысл уточнить
assert(!(CurFile>=FileCount && FileCount && (ReturnCurrentFile||!SelFileCount)));
?

Maximus

2011-01-30 15:07

reporter   bugnote:0006285

наверное так:
assert(!FileCount || !(ReturnCurrentFile||!SelFileCount) || (CurFile<FileCount));

vskirdin

2011-01-31 11:20

administrator   bugnote:0006293

после build 1804

Issue History

Date Modified Username Field Change
2011-01-20 11:08 Maximus New Issue
2011-01-20 21:19 Maximus File Added: 110120.diff.7z
2011-01-20 21:19 Maximus Note Added: 0006227
2011-01-28 06:23 vskirdin Note Added: 0006264
2011-01-28 19:59 Maximus File Added: 110128.diff.7z
2011-01-28 20:00 Maximus Note Added: 0006267
2011-01-29 06:02 vskirdin Note Added: 0006268
2011-01-30 15:07 Maximus Note Added: 0006285
2011-01-31 11:19 vskirdin Assigned To => vskirdin
2011-01-31 11:19 vskirdin Status new => assigned
2011-01-31 11:20 vskirdin Note Added: 0006293
2011-01-31 11:20 vskirdin Status assigned => feedback
2011-12-23 08:53 alexy Build => 1804
2011-12-23 08:53 alexy Status feedback => closed
2011-12-23 08:53 alexy Resolution open => fixed
2011-12-23 08:53 alexy Fixed in Version => 2.0