Share

Liferea

Tracker: Patches

5 Fix for 2861203 - ID: 2875648
Last Update: Comment added ( bunk )

The bug is triggered when nrOfRuleFunctions == 0 and this patch calls
rule_init() when this condition is detected. This works for me but I'm not
at all sure if this is the correct way to fix this.


Lars L ( larslj ) - 2009-10-09 11:08

5

Closed

Fixed

Lars Lindner

None

None

Public


Comments ( 4 )

Date: 2009-11-19 07:45
Sender: bunkProject Admin

Changes released in 1.6.1 and 1.7.2.


Date: 2009-10-12 20:37
Sender: llandoProject Admin

Thanks for retesting! You are right it didn't solve the problem.

Therefore I did the following:
for 1.6: I added your patch to the source
for 1.7: I rewrote the rule.[ch] code to avoid exposing globals or
rule_init() at all

I think this should solve the issue in both branches.


Date: 2009-10-10 14:42
Sender: larslj

I tried your patch for 1.6.0 (rev 5194) and it doesn't work. It does
prevent the segmentation fault, but the combobox in the rule editor is now
completly empty.


Date: 2009-10-10 14:09
Sender: llandoProject Admin

Thanks for analyzing the problem and the patch!

To fix SF #2861203 I added the following solution:

Index: src/ui/rule_editor.c
===================================================================
--- src/ui/rule_editor.c (Revision 5192)
+++ src/ui/rule_editor.c (Arbeitskopie)
@@ -198,7 +198,8 @@

if (gtk_combo_box_get_active_iter (optionmenu, &iter)) {
gtk_tree_model_get (gtk_combo_box_get_model (optionmenu), &iter, 1,
&changeRequest, -1);
- do_ruletype_changed (changeRequest);
+ if (changeRequest)
+ do_ruletype_changed (changeRequest);
}
}

I want to avoid exposing the rule initialization to the dialog code. I
hope you do not mind.

To be released with 1.6.1 and 1.7.2. Please retest!


Attached File ( 1 )

Filename Description Download
liferea_2861203.patch Download

Changes ( 6 )

Field Old Value Date By
status_id Open 2009-11-19 07:45 bunk
allow_comments 1 2009-11-19 07:45 bunk
close_date - 2009-11-19 07:45 bunk
resolution_id None 2009-10-10 14:09 llando
assigned_to nobody 2009-10-10 14:09 llando
File Added 345902: liferea_2861203.patch 2009-10-09 11:08 larslj