Ticket #8477 (closed defect: fixed)
Opened 3 years ago
Last modified 3 years ago
Patchset #22999 break UPNP playback of file greater then 4G on 32 bits
| Reported by: | anonymous | Owned by: | danielk |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.24 |
| Component: | MythTV - UPnP | Version: | 0.23-fixes |
| Severity: | medium | Keywords: | UPNP 32 |
| Cc: | Ticket locked: | yes |
Description
Declaration of the variable nFileSize on line 255 of the file upnpcdstv.cpp was changed from type "long long" to "size_t".
On 32 bits system, size_t cannot hold file size greater then 4G breaking the video playback of most HD recording.
Attachments
Change History
comment:1 Changed 3 years ago by robertm
- Owner changed from dblain to danielk
- Status changed from new to assigned
comment:2 Changed 3 years ago by stuartm
- Priority changed from major to minor
- Milestone changed from 0.23-fixes to unknown
comment:3 Changed 3 years ago by anonymous
I had many problems playing recordings on my PS3 through UPnP. I can confirm that compiling the trunk code with the proposed modification in the bug description (changing the nFileSize variable type from “size_t” to “long long”) fixes my bug. I’m running mythbackend with a hauppauge hd-pvr on a old P4 2.0ghz.
comment:4 Changed 3 years ago by danielk
- Status changed from assigned to closed
- Resolution set to fixed
comment:5 follow-up: ↓ 6 Changed 3 years ago by superm1@…
Could this by chance be backported to 0.23-fixes too?
comment:6 in reply to: ↑ 5 Changed 3 years ago by anonymous
Replying to superm1@…:
Could this by chance be backported to 0.23-fixes too?
I definitively need this to upgrade my system to the 0.23 version. Please include this in the 0.23-fixes (it is due in 9 days!!).
comment:7 Changed 3 years ago by anonymous
- Status changed from closed to new
- Resolution fixed deleted
Please include in 0.23-fixes
comment:8 Changed 3 years ago by robertm
- Status changed from new to closed
- Resolution set to fixed
- Ticket locked set
Reopening closed tickets is not how to get them included.
Daniel, reassigning to you since this was part of the programinfo refactor and david hasn't been around in a while.