Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3825 closed defect (notsugar)

Browse crashes adding a cookie when connecting to an XS the first time

Reported by: greenfeld Owned by: erikos
Priority: Unspecified by Maintainer Milestone: Unspecified
Component: Browse Version: 0.96.x
Severity: Unspecified Keywords: 12.1.0 olpc-test-passed
Cc: dsd Distribution/OS: OLPC
Bug Status: New

Description

  1. Install OLPC 12.1.0 os20 on a XO-1.5.
  2. Connect to a XS network and register to the schoolserver while Browse is NOT opened.
  3. Start Browse for the first time, and click on the "Local Schoolserver" link.
  4. Browse (version 140) will crash due to a segmentation fault that appears to be related to a strcmp() while adding a cookie.
  5. Attempting to restart Browse to go back to the XS usually will work.

Attachments (1)

core.1349.xsfirstconn.txt (20.2 KB) - added by greenfeld 12 years ago.
backtrace

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by greenfeld

backtrace

comment:1 Changed 12 years ago by dsd

  • Keywords 12.1.0 added

comment:2 Changed 12 years ago by dsd

When browse is opened for the first time after registering, it explicitly creates a cookie (from Python code) and adds it to libsoup's cookie storage.

But a missing annotation in libsoup caused this in-memory copy to be trashed soon after the cookie was created, causing the crash seen here.

It didn't occur on subsequent runs, because the cookie *was* correctly saved on disk, and subsequent runs detect that the cookie is already present in the DB so they don't create another one.

https://bugzilla.gnome.org/show_bug.cgi?id=682554

RPM for testing: http://dev.laptop.org/~dsd/20120823/libsoup-2.38.1-2.fc17.dsd1.i686.rpm

comment:4 Changed 12 years ago by greenfeld

The Fedora package fixes the crash; will need an ARM build for XO-1.75 as well.

comment:5 Changed 12 years ago by dsd

  • Keywords olpc-test-pending added
  • Resolution set to notsugar
  • Status changed from new to closed

Will be included in OLPC 12.1.0 build 21.

comment:6 Changed 12 years ago by greenfeld

  • Keywords olpc-test-passed added; olpc-test-pending removed

This is fixed for XO-1.5 and XO-1.75 in OLPC 12.1.0 build 21.

Note: See TracTickets for help on using tickets.