Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#3421 closed Bug (fixed)

crash when opening properties dialog on a magnet link w/o metainfo

Reported by: jch Owned by: charles
Priority: Normal Milestone: 2.12
Component: GTK+ Client Version: 2.11
Severity: Normal Keywords:
Cc:

Description


Attachments (2)

0001-Protect-against-division-by-zero-in-details-window.patch (1.2 KB) - added by jch 12 years ago.
crash.r11336 (5.2 KB) - added by kovalev 11 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 12 years ago by charles

  • Milestone changed from None Set to 2.02
  • Status changed from new to assigned
  • Version changed from 2.01+ to 2.00

Fixed in trunk by r11002

Fixed in the 2.0x branch by r11003

comment:2 Changed 12 years ago by charles

  • Resolution set to fixed
  • Status changed from assigned to closed
  • Summary changed from Crash when double-clicking on incomplete torrent to crash when opening properties dialog on a magnet link w/o metainfo

comment:3 Changed 11 years ago by kovalev

  • Priority changed from High to Normal
  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Version changed from 2.00 to 2.11+

The issue seems has re-appeared in r11336:

Oct 20 23:14:05 kovalev-home kernel: [41025.598138] ratelimit: 43 callbacks suppressed Oct 20 23:14:05 kovalev-home kernel: [41025.598143] transmission-gt[1579]: segfault at 0 ip 000000000047a927 sp 00007fffa81458b0 error 6 in transmission-gtk[400000+e6000]

comment:4 Changed 11 years ago by charles

kovalev, could you attach a gdb backtrace for this? Those kernel messages don't do any good at all, unfortunately.

Changed 11 years ago by kovalev

comment:5 Changed 11 years ago by kovalev

Program received signal SIGSEGV, Segmentation fault. 0x000000000047a927 in tr_truncd (x=-nan(0x8000000000000), decimal_places=0)

at utils.c:1440

1440 *strchr(buf,'.') = '\0';

See full debugger output attached (crash.r11336)

comment:6 Changed 11 years ago by charles

  • Milestone changed from 2.02 to 2.12
  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in trunk by r11343. Thanks kovalev!

comment:7 Changed 11 years ago by charles

  • Version changed from 2.11+ to 2.11

The division by zero is actually present in 2.11 as well, so I'm going to push the version number back to that.

The strchr(buf,'.') line is a separate issue being handled in #3521

comment:8 Changed 11 years ago by charles

  • Keywords backport-2.0x added

comment:9 Changed 11 years ago by charles

  • Keywords backport-2.0x removed
Note: See TracTickets for help on using tickets.