bugdevilspie2 - Bugs: bug #66070, crash and memory leak when...

 
 

bug #66070: crash and memory leak when changing config files

Submitter:  Eric Koldeweij <eric_koldeweij>
Submitted:  Thu 08 Aug 2024 06:48:58 PM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  dsalt
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 17 Oct 2024 04:11:35 PM UTC, comment #1: 

Fix committed for 0.45.

Darren Salt <dsalt>
Group administrator
Thu 08 Aug 2024 06:48:58 PM UTC, original submission:  

When editing config files while dp2 is running, the application frequently crashes. It is best reproducible when (using vim as an editor works great, it creates temporary files which trigger this) opening devilspie2.lua and just write the file. It triggers many reload events, at some point it crashes.
The reason for this is that the list structure is not correctly cleaned up in clear_file_lists() causing a crash if the function is called again before the lists are repopulated. Even if it does not crash memory is lost since the old list is not properly freed.
Attached is a small patch which fixes the issue.

Eric Koldeweij <eric_koldeweij>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #56340:  dp.patch added by eric_koldeweij (374B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dsalt (Posted a comment)
  • -email is unavailable- added by eric_koldeweij (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2024-10-17 dsalt StatusNone Fixed
        Assigned toNone dsalt
        Open/ClosedOpen Closed
    2024-08-08 eric_koldeweij Attached File- Added dp.patch, #56340

    Back to the top

    Powered by Savane 3.14-8eb0.
    Corresponding source code