mate-menu package needs updating

Bug #1427742 reported by Martin Wimpress 
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu MATE
Fix Released
Undecided
Unassigned
mate-menu (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

A new version of mate-menu is available that adds translations and also addresses shell command injection.

  * https://bugs.launchpad.net/ubuntu-mate/+bug/1422402

The source for the packages are available from the following repositories in the 'ubuntu/15.04' branch.

  git clone https://alioth.debian.org/anonscm/git/pkg-mate/mate-menu.git
  cd mate-menu
  git checkout ubuntu/15.04
  debian/rules get-orig-source

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

mate-menu (5.6.2-0ubuntu1) vivid; urgency=medium is in changelog and doesn't appear to have been landed in vivid.

Also, I'm a bit concerned about what some of the entries mean in changelog:
+ + Removed package management features.
+ + Removed useless imports and dead code.
+ + Refactored some os.system() calls to Pythonic equivilents.

Given that it's past Feature Freeze, we need to make sure all the changes are indeed bugfixes, and removal of features sounds a bit like a feature :)

Revision history for this message
Martin Wimpress  (flexiondotorg) wrote :

There is a Shell Command Injection vulnerability in the version of MATE Menu currently residing in the official Ubuntu archive. This issue is described here:

  * https://bugs.launchpad.net/ubuntu-mate/+bug/1422402

mate-menu 5.6.2 directly addresses the issue above, but as you point out was not released in Ubuntu. Should I change the entry for mate-menu 5.6.2 in the changelog to UNRELEASED?

However, after doing a code review I found other exploitable methods in the package management features of MATE Menu.

So I started on mate-menu 5.6.3 and the following changes address the other exploitable code.

  + Removed package management features.
  + Removed useless imports and dead code.
  + Refactored some os.system() calls to Pythonic equivalents.

Personally, I do not think a Menu should be trying to be a package manager, certainly not one that is exploitable. Before removing those features I consulted with the Ubuntu MATE community here:

  * https://plus.google.com/103917631499285627130/posts/jkrMzsC3Brs

The message was clear, most people didn't know the package management features existed and of those that did know about, they didn't use it. So I took the decision to remove an insecure unused feature rather than fix it.

I hope that explains my rationale.

Revision history for this message
Martin Wimpress  (flexiondotorg) wrote :

After discussing with Iain Lane I have consolidated the changelog entry, so the unreleased 5.6.2 version is no longer listed.

Revision history for this message
Martin Wimpress  (flexiondotorg) wrote :

As requested by Didier Roche I have attached a debdiff for the change between 5.6.1 and 5.6.3.

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Perfect, looking good and sponsored, thanks! :)

Changed in mate-menu (Ubuntu):
status: New → Fix Committed
Revision history for this message
Artur Rona (ari-tczew) wrote :

mate-menu (5.6.3-0ubuntu1) vivid; urgency=medium

  [ Martin Wimpress ]
  * New upstream release.
    + Added translations.
    + Fixed shell code injection. Closes (LP: #1422402)
      + Removed package management features.
      + Removed useless imports and dead code.
      + Refactored some os.system() calls to Pythonic equivalents.
    + Refactored calls to the deprecated commands.getouput().
    + Removed unused icons.
    + Added a single, non-distro secific, icon for use everywhere.
    + Fixed lock screen.
  * debian/copyright:
    + Remove COPYING from copyright.
    + Update copyright attribution for new mate-logo.svg.
    + Update copyright attribution for translators.
    + Remove obsolete entries from copyright.

  [ Mike Gabriel ]
  * debian/control:
    + Add to D (mate-menu): libglib2.0-bin (for glib-compile-schema in postinst
      script). (Closes: #779102).
 -- Martin Wimpress <email address hidden> Fri, 06 Mar 2015 22:03:23 +0000

Changed in mate-menu (Ubuntu):
importance: Undecided → Wishlist
status: Fix Committed → Fix Released
Changed in ubuntu-mate:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.