Opened on 05/21/2014 at 11:40:22 AM

Closed on 05/23/2014 at 01:25:39 PM

Last modified on 05/20/2015 at 02:22:39 PM

#530 closed defect (fixed)

patterns.ini isn't being written on Firefox 20 (Permission denied)

Reported by: trev Assignee:
Priority: P1 Milestone: Adblock-Plus-2.6.2-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc:, Blocked By:
Blocking: #153 Platform: Firefox
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):



Adblock Plus 2.6, Firefox 20, Windows 7 x64

How to reproduce

  1. Change anything in the Filter preferences.

Observed behaviour

patterns.ini.tmp file is being written but not renamed into patterns.ini. Error Console shows: "Win error 5 during operation move (Permission denied). The patterns.ini file is stuck in its original state.

Expected behaviour

patterns.ini should be replaced by new data.

Attachments (0)

Change History (11)

comment:1 Changed on 05/21/2014 at 11:43:49 AM by mapx

  • Cc added

comment:2 Changed on 05/21/2014 at 12:55:07 PM by trev

Apparently, this was fixed in Firefox 23, between the nightly builds 2013-04-23 and 2013-04-24. The changes for that day are:

Interestingly, elemhide.css is being overwritten correctly even though it should be using exactly the same code.

comment:3 Changed on 05/21/2014 at 01:30:51 PM by trev

Some more investigation: for some reason, OS.File.move() indeed cannot overwrite that particular patterns.ini file (reproducible from Scratchpad). If I create a copy of that file then this one can be overwritten. All permissions are inherited in both cases, cacls.exe output is identical. Same goes for the files produced with icacl.exe /save, the info saved there is absolutely identical.

comment:4 Changed on 05/21/2014 at 01:57:21 PM by trev

Stupid me, the file is simply locked and cannot be removed because of that (thanks fhd). So our code reading patterns.ini leaves it locked for some reason, this isn't an issue for elemhide.css because we don't read it.

comment:5 Changed on 05/21/2014 at 02:01:06 PM by trev

And that means that bug 858723 is the issue here - I guess we get file ID 0 for patterns.ini which prevents OS.File.close() from working correctly.

comment:6 Changed on 05/21/2014 at 02:49:39 PM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:7 Changed on 05/23/2014 at 01:25:39 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:8 Changed on 05/23/2014 at 01:25:54 PM by trev

  • Milestone set to Adblock-Plus-for-Firefox-next

comment:9 Changed on 06/04/2014 at 10:47:02 AM by jobp

  • Cc added

comment:10 Changed on 07/15/2014 at 12:36:06 PM by trev

  • Blocking 153 added
  • Platform set to Firefox/Firefox Mobile

comment:11 Changed on 05/20/2015 at 02:22:39 PM by philll

  • Platform changed from Firefox/Firefox Mobile to Firefox

Made Firefox and Firefox mobile available as seperate platforms.

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
Note: See TracTickets for help on using tickets.