Bug 16574 - msec: unable to add an exception (TypeError: pack_start() takes exactly 5 arguments (2 given))
Summary: msec: unable to add an exception (TypeError: pack_start() takes exactly 5 arg...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: advisory MGA5-64-OK
Keywords: PATCH, validated_update
Depends on:
Blocks: 11778
  Show dependency treegraph
 
Reported: 2015-08-11 22:27 CEST by papoteur
Modified: 2016-07-08 21:51 CEST (History)
7 users (show)

See Also:
Source RPM: msec
CVE:
Status comment:


Attachments
Syntax corrections (1.12 KB, patch)
2015-08-12 23:11 CEST, papoteur
Details | Diff

Description papoteur 2015-08-11 22:27:16 CEST
Description of problem:
Adding an exception have no effect.
In console, I get:
Traceback (most recent call last):
  File "/usr/share/msec/msecgui.py", line 1417, in add_exception
    return self.exception_changed(None, None, None, model)
  File "/usr/share/msec/msecgui.py", line 1445, in exception_changed
    hbox.pack_start(Gtk.Label(_("Check: "), True, True, 0))
TypeError: pack_start() takes exactly 5 arguments (2 given)


Version-Release number of selected component (if applicable):
msec-gui-1.13

How reproducible:
Launch msecgui
Tabs "Security settings"/"Exceptions"
Click button "Add a rule"
Nothing happens in interface. 
Get the error in console.



Reproducible: 

Steps to Reproduce:
Comment 1 papoteur 2015-08-12 18:49:32 CEST
In msecgui.py, these lines should be modified in:

l 1445:         hbox.pack_start(Gtk.Label(_("Check: ")), True, True, 0)
l 1460:       hbox.pack_start(Gtk.Label(_("Exception: ")), True, True, 0)
I will prepare a patch.
Comment 2 papoteur 2015-08-12 23:11:41 CEST
Created attachment 6923 [details]
Syntax corrections

Correction in the parathensis repartition in pack_start command
Comment 3 Thierry Vignaud 2015-08-24 10:51:54 CEST
Another fallout from the autoconversion :-(
Please commit & release a new msec

Keywords: (none) => PATCH
CC: (none) => thierry.vignaud, yves.brungard_mageia
Source RPM: (none) => msec

Thierry Vignaud 2015-08-24 10:52:13 CEST

Summary: msec: unable to add an exception => msec: unable to add an exception (TypeError: pack_start() takes exactly 5 arguments (2 given))

Thierry Vignaud 2015-08-24 10:52:35 CEST

Blocks: (none) => 11778

Comment 4 Mageia Robot 2015-08-28 20:26:39 CEST
commit 12b1f5d8b2ecdb1a5d4b99a03e0a119f3228c45f
Author: SARL ENR 68 <david@...>
Date:   Fri Aug 28 20:26:24 2015 +0200

    Correction bug 16574 unable to add an exception (mga#16574)
    
    - by papoteur: https://bugs.mageia.org/attachment.cgi?id=6923
---
 Commit Link:
   http://gitweb.mageia.org/software/msec/commit/?id=12b1f5d8b2ecdb1a5d4b99a03e0a119f3228c45f
Comment 5 David GEIGER 2015-08-28 20:40:59 CEST
@ Thierry:

All changes from papoteur are now committed, so I guess we go now on a "2.0" version of msec?

What is the command that you use for create a new tag/release on mageia git?

$ git tag -a 2.0 -m 'release 2.0' 12b1f5d8b2ecdb1a5d4b99a03e0a119f3228c45f
$ git push origin 2.0

CC: (none) => geiger.david68210

Comment 6 andré blais 2015-12-20 04:23:58 CET
The bug had erased all my exceptions.
The patch works nicely.
(although the error msg was misleading, saying "2 arguments instead of 5",
whereas it was 1 argument instead of 4.)

If ever the lines 1308 1313 1318 and/or 1333 are uncommented,
 hbox.pack_start(button, False, 0)
s/b
 hbox.pack_start(button, False, False, 0)

CC: (none) => andre999mga

Comment 7 w unruh 2016-07-04 06:38:20 CEST
Still broken  almost a year later.

msec-1.15-1.mga5.x86_64.rpm

CC: (none) => unruh

Comment 8 w unruh 2016-07-04 06:56:35 CEST
Ie, the patch works, but the patch is not included in 1.15 released Apr 2016.
Comment 9 Mageia Robot 2016-07-04 09:09:53 CEST
commit e04b7e22dd52cbb8c6e1d910b55f2927a4c47984
Author: Papoteur <papoteur@...>
Date:   Mon Jul 4 09:03:37 2016 +0200

    Correction unable to add an exception (mga#16574)
---
 Commit Link:
   http://gitweb.mageia.org/software/msec/commit/?id=e04b7e22dd52cbb8c6e1d910b55f2927a4c47984
Comment 10 David GEIGER 2016-07-04 19:59:34 CEST
msec 1.16 submitted and uploaded on mga5 Core/Updates_testing repo.
Comment 11 papoteur 2016-07-04 20:28:38 CEST
See the comment 1 to know how to try the update.

Advisory

The feature "Add an exception" didn't work.
The update gives a solution to this bug.

x86_64
msec-1.16-1.mga5.x86_64.rpm
msec-gui-1.16-1.mga5.x86_64.rpm

i586
msec-1.16-1.mga5.i586.rpm
msec-gui-1.16-1.mga5.i586.rpm

SRPMS
msec-gui-1.16-1.mga5.src.rpm

Assignee: bugsquad => qa-bugs
Whiteboard: (none) => advisory

Comment 12 James Kerr 2016-07-08 12:43:09 CEST
Testing on mga5-64

Installed from testing:
msec-gui-1.16-1.mga5
msec-1.16-1.mga5

Created a rule and verified that it was correctly added to /etc/security/msec/exceptions

OK for mga5-64

CC: (none) => jim
Whiteboard: advisory => advisory MGA5-64-OK

Comment 13 claire robinson 2016-07-08 16:52:00 CEST
Validating
claire robinson 2016-07-08 16:52:17 CEST

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs

Comment 14 Mageia Robot 2016-07-08 21:51:41 CEST
An update for this issue has been pushed to the Mageia Updates repository.

http://advisories.mageia.org/MGAA-2016-0102.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.