[Tracker] crash in Model::StatChanged ()

[diver]

I tried to zip Desktop folder using:

zip -yr Desktop.zip Desktop/

I have an opened query window for "zip"
As soon as i executed this command tracker crashed.
See screenshot
hrev31814 in virtualbox

[bonefish]

Also encountered in hrev32727 alpha 1 branch.

00114b90 <BPrivate::Model::StatChanged(void)>:
  114b90:       55                      push   %ebp
  114b91:       89 e5                   mov    %esp,%ebp
  114b93:       83 ec 0c                sub    $0xc,%esp
  114b96:       57                      push   %edi
  114b97:       56                      push   %esi
  114b98:       53                      push   %ebx
  114b99:       e8 00 00 00 00          call   114b9e <BPrivate::Model::StatChanged(void)+0xe>
  114b9e:       5b                      pop    %ebx
  114b9f:       81 c3 a6 c4 0c 00       add    $0xcc4a6,%ebx
  114ba5:       8b 7d 08                mov    0x8(%ebp),%edi
  114ba8:       8b 77 1c                mov    0x1c(%edi),%esi
  114bab:       83 c4 f8                add    $0xfffffff8,%esp
  114bae:       8b 57 74                mov    0x74(%edi),%edx
  114bb1:       8b 4a 10                mov    0x10(%edx),%ecx

The last instruction is the point of crash; %edx is NULL. %edi should be the object pointer and 0x74(%edi) the fNode member. So apparently the Model's fNode member is NULL.

I've already seen the bug several times. It usually happens with a tracker window for a directory in which a lot of file adding/removing is happing. Easy to reproduce with:

mkdir /tmp/tt; cd /tmp/tt; open .; while true; do touch $(seq 100); rm $(seq 100); done

[aldeck]

Thanks for the test case. Very well reproducible indeed. On it

[aldeck]

Fixed in hrev33744

[aldeck]

(In #4831) Duplicate of #4130, fixed by hrev33744.

for reference, the last bug i wanted to refer to in the previous comment was #4271.

[aldeck]

(In #4271) Most certainly a duplicate of #4130, fixed in hrev33744. Please test

[aldeck]

(In #4610) Most probably a duplicate of #4130, fixed by hrev33744. Please test

[aldeck]

The fix in hrev33744 wasn't correct as it would stop to manage moved/renamed files. More correct fix in hrev33768

[aldeck]

sorry, previous comment should have said hrev33769

[aldeck]

(In #4845) Ok, then chances are very high that this is a duplicate of #4130. Just reopen if you see it again on revisions > hrev33769. Thanks for testing. Btw i'm curious what is that scripting tool you are using?