Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 2 years ago

Closed 2 years ago

#10221 closed Bug Report - Crash (Fixed)

Segfault in MediaMonitor following 027352d62

Reported by: stuartm Owned by: beirdo
Priority: blocker Milestone: 0.25
Component: MythTV - General Version: Master Head
Severity: high Keywords:
Cc: Ticket locked: no

Description

Following [027352d62] plugging in a USB storage device causes the frontend to crash.

Backtrace + Log - http://pastebin.com/zjAy8xE6

Attachments (0)

Change History (4)

comment:1 Changed 2 years ago by Lawrence Rust <lvr@…>

I think that this looks more like a problem with memory corruption. Certainly the backtrace shows the exception is in MythMediaDevice::performMountCmd (called from the patched MediaMonitor::CheckDevices?) but the failing line is a LOG output and Qt is simply formatting a string.

Earlier in the log are numerous lines showing the attempted mount command without problem. Would be useful to see a valgrind run.

comment:2 Changed 2 years ago by Github

Do not keep trying to mount when it fails

Refs #10221

If a USB stick (or other removable drive) is attached to the frontend, and the
user running the frontend is not able to mount it with "mount /dev/sdc1" or the
like, we do not want to keep trying (twice a second!) to mount it. There's
no purpose to that other than extra load, and filling log files with useless
crap.

Branch: master
Changeset: 1193d2468e7bb110c146aace9c6b24e7d7ac6184

comment:3 Changed 2 years ago by beirdo

  • Status changed from new to infoneeded_new

I can not reproduce the crash. However, the repeated mounting is now gone. If you could retest and see if this does it for you, I'd appreciate it.

Also, just a note: for the QDbus stuff to work, (in ubuntu at least), you need the "dbus" and "udisks" packages both installed.

comment:4 Changed 2 years ago by stuartm

  • Resolution set to Fixed
  • Status changed from infoneeded_new to closed

Without the repeated attempts to mount the segfault is no longer repeatable.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.