Opened 12 years ago

Closed 12 years ago

Last modified 8 years ago

#1028 closed defect (fixed)

crash when installing content bundles

Reported by: dsd Owned by: erikos
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Sugar Version: Unspecified
Severity: Unspecified Keywords: r+
Cc: Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

For the standard content bundles already in wide distribution, ContentBundle.get_bundle_id() returns None, resulting in sugar sending a None value over DBus causing sugar to crash.

Here's a patch to fix it

Attachments (1)

0001-Fix-ContentBundle-bundle_id.patch (2.5 KB) - added by dsd 12 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by dsd

  • Keywords r? added

comment:2 Changed 12 years ago by erikos

  • Keywords r+ added; r? removed

Thanks, looks good. We can apply to the master branch and as well for 0.84. Can you state a quick test case for that? (link to a bundle)

Thanks

comment:3 Changed 12 years ago by dsd

yeah, just click this link inside sugar to reproduce: http://wiki.laptop.org/images/3/34/ICDL-8.xol

comment:4 Changed 12 years ago by dsd

  • Resolution set to fixed
  • Status changed from new to closed

pushed to master and sucrose-0.84

comment:5 Changed 8 years ago by dnarvaez

  • Component changed from sugar-toolkit to Sugar
Note: See TracTickets for help on using tickets.