Opened 13 years ago

Closed 4 years ago

#9655 closed Bug Report - General (Fixed)

mythvideo fails and locks when playing filename that begins with "#"

Reported by: Ryan Patterson <ryan.goat@…> Owned by: robertm
Priority: minor Milestone: 31.0
Component: MythTV - General Version: 0.24-fixes
Severity: medium Keywords: filename
Cc: Ticket locked: no

Description

Trying to use mythvideo to play a file with the name "#1 Cheerleader Camp.mpg" results in mythfrontend locking up at the "please wait" notification. Changing the file name to "1# Cheerleader Camp.mpg" resolves the error. So the error is only triggered by the filename beginning with the "#" character.

Maybe other special characters also trigger this error. I have not tested.

I will attempt to attach the log output from mythfrontend. But this should be easy to reproduce.

My results are with Ubuntu 10.10 MythTV 0.24-fixes compiled by me from git with no extra patches. Mythvideo uses storage groups.

Attachments (4)

error.log (10.8 KB) - added by Ryan Patterson <ryan.goat@…> 13 years ago.
mythfrontend log showing the error playing file: #1 Cheerleader Camp.mpg
hasfragment_v1.diff (944 bytes) - added by greg 13 years ago.
Fix for # in a myth URL
mythfrontend.log (662 bytes) - added by iskimj@… 7 years ago.
fail with directory "#1 Videos"
version.txt (951 bytes) - added by iskimj@… 7 years ago.
version output

Download all attachments as: .zip

Change History (11)

Changed 13 years ago by Ryan Patterson <ryan.goat@…>

Attachment: error.log added

mythfrontend log showing the error playing file: #1 Cheerleader Camp.mpg

comment:1 Changed 13 years ago by robertm

Component: Plugin - MythVideoMythTV - General

Not a MythVideo? specific error, this will happen with any URL with a # at the beginning in any part of Myth as it breaks URL parsing.

comment:2 Changed 13 years ago by robertm

Type: Bug Report - CrashBug Report - General

Also, this is not a crash.

Changed 13 years ago by greg

Attachment: hasfragment_v1.diff added

Fix for # in a myth URL

comment:3 Changed 13 years ago by greg

Ryan: Please try the attached patch hasfragment_v1.diff

comment:4 in reply to:  3 Changed 13 years ago by Ryan Patterson <ryan.goat@…>

Replying to greg:

Ryan: Please try the attached patch hasfragment_v1.diff

I applied the patch to 0.24-fixes, recompiled and changed the filename back to the one myth had the problem with. The video now plays without issue. Thanks.

comment:5 Changed 13 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: newclosed

This patch fixes #9655 by trying to properly handle filenames with a '#' in them.

Branch: master Changeset: e801178eb85018819a890d88b7761cc3d10544f1

Changed 7 years ago by iskimj@…

Attachment: mythfrontend.log added

fail with directory "#1 Videos"

Changed 7 years ago by iskimj@…

Attachment: version.txt added

version output

comment:6 Changed 7 years ago by iskimj@…

Resolution: fixed
Status: closednew

I am experiencing this bug in 0.28 exactly as OP described. A video in a directory "#1 Videos" fails but if the directory name is changed to "1# Videos" it works. I've attached relevent log file section and version output.

comment:7 Changed 4 years ago by Stuart Auchterlonie

Milestone: 0.2531.0
Resolution: Fixed
Status: newclosed

Retested with master and this works fine

Note: See TracTickets for help on using tickets.