[Hardy] The Prism Refractor Extension (v0.2) causes file downloads to kill firefox 3

Bug #247132 reported by 20after4 on 2008-07-10
4
Affects Status Importance Assigned to Milestone
xulrunner-1.9 (Ubuntu)
High
Alexander Sack
Hardy
High
Alexander Sack
Intrepid
High
Alexander Sack

Bug Description

Binary package hint: firefox

Tested with Ubuntu 8.04 running Firefox 3, all OS updates installed.

After installing the Prism Refractor extension (https://addons.mozilla.org/en-US/firefox/addon/6665 ) - downloading any file (consistently) kills the browser after the download completes.

This is most likely caused by a bug in the Prism extension, however, it only happens with Ubuntu's Firefox 3 package. I can't trigger the bug when using the vanilla Firefox 3 tarball from mozilla.org ( same system, same Firefox profile.)

I already reported this bug in the Mozilla bug tracker: https://bugzilla.mozilla.org/show_bug.cgi?id=444472

Since it's related to Ubuntu's version of Firefox I decided to post it here as well. Refer to the Bugzilla link for detailed explanation.

Alexander Sack (asac) wrote :

after debugging with prism developer we found that there are two bugs here:

 1. prism has obsolete code that hooks in the download manager - which it shouldnt do. prism author will fix this
 2. our patch for https://bugzilla.mozilla.org/show_bug.cgi?id=368428 lacks a NULL check in OpenFile -> we will fix this in update.

Changed in firefox:
importance: Undecided → High
status: New → Triaged
assignee: nobody → asac
importance: Undecided → High
status: New → Triaged
Changed in xulrunner-1.9:
assignee: nobody → asac
Alexander Sack (asac) wrote :

committed fix to xulrunner-1.9.head branch

Changed in xulrunner-1.9:
status: Triaged → Fix Committed
20after4 (twentyafterfour) wrote :

Awesome, good work Alexander!

Steve Langasek (vorlon) wrote :

Accepted into -proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in xulrunner-1.9:
status: Triaged → Fix Committed

On Sat, Jul 12, 2008 at 10:33:28PM -0000, Steve Langasek wrote:
> Accepted into -proposed, please test and give feedback here. Please see
> https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to
> enable and use -proposed. Thank you in advance!
>
> ** Changed in: xulrunner-1.9 (Ubuntu Hardy)
> Status: Triaged => Fix Committed
>
> ** Tags added: verification-needed
>

Thanks Steve,

anyone: please test and comment even if you verified the PPA packages
before.

Thanks in advance,

 - Alexander

Launchpad Janitor (janitor) wrote :
Download full text (3.7 KiB)

This bug was fixed in the package xulrunner-1.9 - 1.9.0.1+build1+nobinonly-0ubuntu1

---------------
xulrunner-1.9 (1.9.0.1+build1+nobinonly-0ubuntu1) intrepid; urgency=low

  * LP: #247494 - new upstream stability/security release (v1.9.0.1 build1)
    - see USN-626-1

  [ Fabien Tassin <email address hidden>]
  * Add a build-system for xulrunner application inside the SDK.
    mozilla-devscripts is able to make use of this
    - add debian/create-build-system.sh
    - update debian/rules
  * Rename the ld.so.conf.d file to xulrunner-1.9.conf as it seems
    extension matters
    - update debian/rules
    - update debian/xulrunner-1.9.postinst
  * Get DEB_MOZ_VERSION and DEB_MOZ_VERSION from changelog and make more use
    of variables to make the merge with the 1.9.1 branch easier
    - update debian/rules
    - update debian/xulrunner-1.9.postinst
    - update debian/xulrunner-1.9.postrm
  * Make EM_TRANSLATION_VERSION follow upstream version now that strings are
    hard frozen and bump EM_TRANSLATION_MAX_VERSION to 1.9.0.*
    - update debian/rules

  [ Alexander Sack <email address hidden> ]
  * borrow lockPref patch from debian xulrunner (Debbugs: #469020)
    - add debian/patches/bzXXX_deb469020_lockPref_everywhere.patch
    - update debian/patches/series
  * Debian compatibility patch that is supposed to make xulrunner also
    consider /usr/lib/mozilla/plugins/ in sid as of xulrunner 1.9~rc2-4
    - add debian/patches/bzXXX_sysplugin_support.patch
    - update debian/patches/series
  * prepatch fix for bmo: #120380 - 'needsterminal flag in mailcap
    must be respected'
    - add debian/patches/bz120380_att326044.patch
    - update debian/patches/series
  * add xre part missed by debian for sysplugin support
    - add debian/patches/bzXXX_sysplugin_support_xre_part.patch
    - update debian/patches/series
  * drop patches applied upstream
    - delete debian/patches/bz428848_att319775_fix_venkman_chrome_access.patch
    - update debian/patches/series
  * housekeeping for debian/patches directory; remove obsolete patches from
    debian/patches/ and drop commented patches from series
    - delete debian/patches/bz384304_fix_recursive_symlinks.patch
    - delete debian/patches/bzr423334_att310581_leak_initparser.patch
    - debian/patches/drop_bz418016.patch
    - update debian/patches/series
  * (disabled in intrepid) fix "jemalloc not enabled in --with-xul-sdk= builds": we
    fix this by building libjemalloc as a static lib and linking xulrunner-bin and
    xulrunner-stub against it.
    - add debian/patches/jemalloc_in_xul.patch
    - add debian/patches/jemalloc_static.patch
    - update debian/patches/series
    - update debian/rules
    - update debian/xulrunner-1.9.postinst
  * link nss/nspr include directories to xulrunner-1.9 sdk in order to
    allow upstream extensions to be built against ubuntu xulrunner.
    - add debian/xulrunner-1.9-dev.links
  * add empty xulrunner-dev package to ease sync/merge tasks for ubuntu
    by providing the package name used by debian.
    - update debian/control
  * fix makefile style variable eval in xulrunner-1.9 prerm script and use
    proper sh'ish style
    - update debian/xulrunner-1.9.prerm
 ...

Read more...

Changed in xulrunner-1.9:
status: Fix Committed → Fix Released
Martin Pitt (pitti) wrote :

Copied to hardy-updates.

Changed in xulrunner-1.9:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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