wine opens the system to null pointer dereference security bugs (vm.mmap_min_addr)

Bug #475540 reported by Laurent Bonnaud
266
This bug affects 2 people
Affects Status Importance Assigned to Milestone
wine (Ubuntu)
Fix Released
High
Kees Cook

Bug Description

Binary package hint: wine

The wine package installs this file:

  /etc/sysctl.d/wine.sysctl.conf

with the following contents:

  # Wine needs to access the bottom 64k of memory in order to launch
  # 16 bit programs.
  vm.mmap_min_addr = 0

This is a problem because it makes the system vulnerable to kernel null pointer dereference exploits.

I guess that the package does this to offer a "works out of the box" experience. Unfortunately it also make the system "unsecure by default". My preference would be "secure by default". The "does not work out of the box" problem is IMHO minor because:

 - it only concerns Win16 binaries and not Win32 binaries
(see http://wiki.debian.org/mmap_min_addr )
 - wine could display a clear error message to the rare users who want to run Win16 binaries

ProblemType: Bug
Architecture: i386
Date: Thu Nov 5 16:55:19 2009
DistroRelease: Ubuntu 9.10
Package: wine 1.0.1-0ubuntu8
ProcVersionSignature: Ubuntu 2.6.31-14.48-generic-pae
SourcePackage: wine
Uname: Linux 2.6.31-14-generic-pae i686
XsessionErrors:
 (polkit-gnome-authentication-agent-1:4006): GLib-CRITICAL **: g_once_init_leave: assertion `initialization_value != 0' failed
 (gnome-settings-daemon:4036): GLib-CRITICAL **: g_propagate_error: assertion `src != NULL' failed

Related branches

Revision history for this message
Laurent Bonnaud (laurent-bonnaud) wrote :
security vulnerability: no → yes
Changed in wine (Ubuntu):
status: New → Confirmed
importance: Undecided → High
Kees Cook (kees)
Changed in wine (Ubuntu):
assignee: nobody → Kees Cook (kees)
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package wine - 1.0.1-0ubuntu9

---------------
wine (1.0.1-0ubuntu9) lucid; urgency=low

  * debian/wine.{postinst,preinst,postrm}: clean up old static sysctl files.
  * debian/control, debian/wine.{templates,config,postinst,postrm}: add
    debconf question for selecting a sensible mmap_min_addr system setting
    (LP: #475540).
  * debian/rules: support "parallel=N" in DEB_BUILD_OPTIONS.
 -- Kees Cook <email address hidden> Sat, 12 Dec 2009 10:28:27 -0800

Changed in wine (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public Security information  
Everyone can see this security related information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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