Opened 12 years ago

Closed 12 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

Change History (4)

comment:1 Changed 12 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 12 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 12 years ago by beirdo

Status: newinfoneeded_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 12 years ago by stuartm

Resolution: Fixed
Status: infoneeded_newclosed

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

Note: See TracTickets for help on using tickets.