Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#11902 closed Bug Report - General (Fixed)

Backend crashes after running HouseKeeperTask 'RecordedArtworkUpdate'

Reported by: Marco Wallerstein <marco.wallerstein@…> Owned by:
Priority: minor Milestone: unknown
Component: MythTV - Mythmetadatalookup Version: 0.27-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

I am running MythTV 0.27. Up to today everything was fine. But now the Backend is constantly crashing since the latest fix update installation. Having a look into the logs shows that it most obviously has an issue with the housekeepertask "RecordedArtworkUpdate?". There is a kind of memory access error happening.

Exact last lines of mythbackend log shows: 2013-10-10 22:57:02.798351 I Queueing HouseKeeperTask? 'RecordedArtworkUpdate?'. 2013-10-10 22:57:02.800995 I Running HouseKeeperTask? 'RecordedArtworkUpdate?'. 2013-10-10 22:57:02.801051 I Child PID 10800 killed with Beendet 2013-10-10 22:57:02.839763 I Performing Artwork Refresh: /usr/bin/mythmetadatalookup --refresh-all-artwork --verbose general --loglevel info 2013-10-10 22:57:02.839867 C Received Speicherzugriffsfehler: Code 1, PID 32, UID 0, Value 0x00000000 2013-10-10 22:57:03.418381 I MainServer::ANN Playback 2013-10-10 22:57:03.418387 I adding: homeserver as a client (events: 0) 2013-10-10 22:57:03.419497 I MainServer::ANN Monitor 2013-10-10 22:57:03.419518 I adding: homeserver as a client (events: 1) Speicherzugriffsfehler (Speicherabzug geschrieben)

I posted the issue in Google+ community already. Got the reply that codelines in https://github.com/MythTV/mythtv/commit/e59e5f63de5f5c3fda1b58b2cf8e370c96c6337c or https://github.com/MythTV/mythtv/commit/3798a07098272be9f04902ec71874b0d2fb1308d might could have caused the issue since they were recently updated.

Once time I even got a more detailed log message which I attach as file.

Attachments (2)

mythbug more detailed log.txt (21.1 KB) - added by Marco Wallerstein <marco.wallerstein@…> 5 years ago.
Detailed error log
mythbackend version.txt (868 bytes) - added by Marco Wallerstein <marco.wallerstein@…> 5 years ago.
mythbackend version output

Download all attachments as: .zip

Change History (8)

Changed 5 years ago by Marco Wallerstein <marco.wallerstein@…>

Detailed error log

Changed 5 years ago by Marco Wallerstein <marco.wallerstein@…>

Attachment: mythbackend version.txt added

mythbackend version output

comment:1 Changed 5 years ago by stuartm

Resolution: Fixed
Status: newclosed

comment:2 Changed 5 years ago by superm1@…

Just so it's clear, this was closed and the commit reverted: https://github.com/MythTV/mythtv/commit/cd6df2c8007913680bdb90f544d5a6b81f560ffa

comment:3 Changed 5 years ago by Raymond Wagner <rwagner@…>

In c712ed04255fe416d95a674ed50e045d864759e8/mythtv:

Add secondary check to prevent housekeeper task from running twice.

This adds an extra check to prevent a HouseKeeperTask? from running
more than once simultaneously. This prevents an issue seen in a
misconfigured child task that was queued in concurrent passes. One
instance had not yet finished when the second one started a minute
later, and overwrote a shared pointer, eventually resulting in a
segmentation fault.

Cause spotted by Jonatan Lindblad.

Refs #11902

comment:4 Changed 5 years ago by Raymond Wagner <rwagner@…>

In 7905afdd4fc7d9ee2eb1bf97bffe0f6e003cc1e9/mythtv:

Add back ArtworkTask? enable check... properly.

This adds the check for DailyArtworkUpdates? back to ArtworkTask?,
following its reversion in ca3a79bf407, but properly runs the
PeriodicHouseKeeperTask? check so it does not requeue once per minute.

Refs #11902

comment:5 Changed 5 years ago by Raymond Wagner <rwagner@…>

In e72f3375f4176d3ed12eb91ce7732143860fdf19/mythtv:

Add secondary check to prevent housekeeper task from running twice.

This adds an extra check to prevent a HouseKeeperTask? from running
more than once simultaneously. This prevents an issue seen in a
misconfigured child task that was queued in concurrent passes. One
instance had not yet finished when the second one started a minute
later, and overwrote a shared pointer, eventually resulting in a
segmentation fault.

Cause spotted by Jonatan Lindblad.

Refs #11902
(cherry picked from commit c712ed04255fe416d95a674ed50e045d864759e8)

comment:6 Changed 5 years ago by Raymond Wagner <rwagner@…>

In 9bfed2b0989f92dc4f98401c968a59ae9238da5d/mythtv:

Add back ArtworkTask? enable check... properly.

This adds the check for DailyArtworkUpdates? back to ArtworkTask?,
following its reversion in ca3a79bf407, but properly runs the
PeriodicHouseKeeperTask? check so it does not requeue once per minute.

Refs #11902
(cherry picked from commit 7905afdd4fc7d9ee2eb1bf97bffe0f6e003cc1e9)

Note: See TracTickets for help on using tickets.