Previews loading more than 454 lines of description text crash Unity

Bug #1066788 reported by David Callé
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Nux
Fix Released
Undecided
Unassigned
Unity
Fix Released
Undecided
Nick Dedekind
nux (Ubuntu)
Fix Released
Undecided
Unassigned
unity (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Working preview:
title = 'test title'
description = 'test line\n' * 454
Unity.GenericPreview.new(title, description, None)

Crashing preview:
title = 'test title'
description = 'test line\n' * 455
Unity.GenericPreview.new(title, description, None)

WARN 2012-10-15 11:10:32 unity.dash.previews.preview Preview.cpp:53 Unable to create Preview object
Segmentation fault (core dumped)

Doing this previously resulted in a black square on the right pane (see https://bugs.launchpad.net/unity/+bug/1039437 ), since the addition of the spinner for previews taking a long time to open, Unity segfaults.

Related branches

Revision history for this message
Omer Akram (om26er) wrote :

Please attach a stacktrace of the crash ;-)

Changed in unity:
status: New → Incomplete
Revision history for this message
David Callé (davidc3) wrote :
Revision history for this message
Albert Astals Cid (aacid) wrote :

Set to new as stacktrace has been provided

Changed in unity:
status: Incomplete → New
Changed in unity:
assignee: nobody → Nick Dedekind (nick-dedekind)
Changed in unity:
status: New → Confirmed
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

Problem occurs because openGL's texture dimension limit is 4096px, and the texture creation failure this is not handled cleanly.

Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

Requires fixes to both nux and unity.

Changed in nux:
status: New → Fix Committed
Changed in unity:
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 6.12.0daily12.12.05-0ubuntu1

---------------
unity (6.12.0daily12.12.05-0ubuntu1) raring; urgency=low

  [ Michael Terry ]
  * debian/control:
    - Update Vcs-Bzr
  * Automatic snapshot from revision 2905 (bootstrap)
    - Hide launcher tooltip once shortcut overlay appears (LP: #1035860)
    - Update shortcut overlay design (LP: #1049601)
    - Don't cut off shadow on launcher icon drag window (LP: #765715)
    - Provide more immediate feedback when clickong on launcher (LP: #893140)
    - Decorate spread windows in switcher (LP: #838222)
    - Make sure to initialize a variable before it can be used (LP: #1073516)
    - Fix FTBFS due to BAMF API change (LP: #1077937)
    - Run gtest during build (LP: #133637)
    - Make shopping lens test more robust (LP: #1077131)
    - Fix some random autopilot test failures (LP: #1078448)
    - Fix autopilot tests for details mode (LP: #1073892)
    - Fix autopilot tests for indicator-session (LP: #1075350)
    - Fix autopilot tests for non-English locales (LP: #1075898)

  [ Didier Roche ]
  * debian/control:
    - adding libgtest-dev and google-mock build-dep now that there are in main
      (LP: #1076763)
    - add libunity-core -> unity-common dep (LP: #1079132)
    - do not have unity-common dep on compiz-gnome: the migration code is only
      ran if compiz-gnome is installed, but compiz-gnome is pulled by compiz
      directly. We don't care if the migration code isn't executed.

  [ bschaefer ]
  * Ported to nux 4.0

  [ Ricardo Salveti de Araujo ]
  * Doesn't need to depend on gcc 4.6 anymore (LP: #1044836)

  [ Nick Dedekind ]
  * Previews loading more than 454 lines of description text crash Unity
    (LP: #1066788)

  [ Brandon Schaefer ]
  * unity::TimeUtil::TimeDelta returns an int value which overflows
    after 24 days of uptime (LP: #806248)
  * Top panel "tooltip" is gone (regression) (LP: #1085045)
  * Dash - Implement *overlay scrollbars* in Dash. Existing scrollbars
    are too thin to grab on to. (LP: #608124)

  [ Andrea Azzarone ]
  * Dash - Regression: Clicking between filter label and 'All' button no
    longer toggles filter label (LP: #1080534)

  [ Didier Roche ]
  * libunity-core fails without the schemas from unity-common (LP:
    #1079132)

  [ Manuel de la Pena ]
  * There is no common ui element that behaves like a url link (LP:
    #1067705)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 2956
 -- Automatic PS uploader <email address hidden> Wed, 05 Dec 2012 09:27:29 +0000

Changed in unity (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nux - 4.0.0daily13.01.11-0ubuntu1

---------------
nux (4.0.0daily13.01.11-0ubuntu1) raring; urgency=low

  [ Didier Roche ]
  * debian/control:
    - put the -common package to any as ppa are regularly failing for other
      archs than i386 to install the -dev package pulling the lib pulling
      -common because of version mismatch

  [ Nick Dedekind ]
  * Previews loading more than 454 lines of description text crash Unity
    (LP: #1066788)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 745
 -- Automatic PS uploader <email address hidden> Fri, 11 Jan 2013 04:02:07 +0000

Changed in nux (Ubuntu):
status: New → Fix Released
Stephen M. Webb (bregma)
Changed in nux:
status: Fix Committed → Fix Released
Changed in unity:
status: Fix Committed → 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.