SabreTools/MPF
1
0
Fork 0
mirror of https://github.com/SabreTools/MPF.git synced 2026-02-08 13:49:48 +00:00
Code Issues 392 Packages Projects Releases 20 Wiki Activity

[Problem] dual-layer PS4 and XBOXONE discs detected as single-layer, PIC not populated in !submissionInfo.txt #444

New Issue
Closed
opened 2026-01-29 16:16:32 +00:00 by claunia · 5 comments
claunia commented 2026-01-29 16:16:32 +00:00
Owner
Copy Link

Originally created by @tjanas on GitHub (Aug 26, 2022).

Originally assigned to: @mnadareski on GitHub.

Version
What version are you using?

  • Stable release (version here)
  • WIP release (2.3-907)

Build
What runtime version are you using?

  • .NET Framework 4.7.2 running on (Operating System)
  • .NET Framework 4.8 running on (Windows 7 Home Premium, 64-bit)
  • .NET Core 3.1 running on (Operating System)

Describe the issue
When dumping a dual-layer (BD-50) PS4 disc, the Ringcodes dialog box does not show entries for both inner and outer layers. Instead, it shows a single Mastering Ring entry for Data Side and Label Side.

Also, the !submissionInfo.txt file that is produced does not contain PIC bytes extracted from name_PIC.bin, and the Media Type is incorrectly detected as BD-ROM-25 instead of BD-ROM-50. See attached.

To Reproduce
Steps to reproduce the behavior:

  1. Use MPF to dump a Sony PlayStation 4 disc that has two layers.
  2. Populate Disc Information when prompted.
  3. Click Accept.

Expected behavior
Media Type should be detected correctly, Ringcodes dialog box should contain fields for both data side layers, and !submissionInfo.txt should contain data for PIC.

Screenshots
image

Additional context
Add any other context about the problem here.
!submissionInfo.txt

Originally created by @tjanas on GitHub (Aug 26, 2022). Originally assigned to: @mnadareski on GitHub. **Version** What version are you using? - [ ] Stable release (version here) - [X] WIP release (2.3-907) **Build** What runtime version are you using? - [ ] .NET Framework 4.7.2 running on (Operating System) - [X] .NET Framework 4.8 running on (Windows 7 Home Premium, 64-bit) - [ ] .NET Core 3.1 running on (Operating System) **Describe the issue** When dumping a dual-layer (BD-50) PS4 disc, the Ringcodes dialog box does not show entries for both inner and outer layers. Instead, it shows a single Mastering Ring entry for Data Side and Label Side. Also, the !submissionInfo.txt file that is produced does not contain PIC bytes extracted from name_PIC.bin, and the Media Type is incorrectly detected as BD-ROM-25 instead of BD-ROM-50. See attached. **To Reproduce** Steps to reproduce the behavior: 1. Use MPF to dump a Sony PlayStation 4 disc that has two layers. 2. Populate Disc Information when prompted. 3. Click Accept. **Expected behavior** Media Type should be detected correctly, Ringcodes dialog box should contain fields for both data side layers, and !submissionInfo.txt should contain data for PIC. **Screenshots** ![image](https://user-images.githubusercontent.com/20304619/186818212-c0993608-c2a2-4be5-9adb-0c902cea5e58.png) **Additional context** Add any other context about the problem here. [!submissionInfo.txt](https://github.com/SabreTools/MPF/files/9429733/submissionInfo.txt)
claunia added the bug label 2026-01-29 16:16:32 +00:00
claunia commented 2026-01-29 16:16:34 +00:00
Author
Owner
Copy Link

@tjanas commented on GitHub (Aug 26, 2022):

Last of Us, The - Part II (USA) (En,Fr,Es,Pt) (Data Disc)_PIC.zip

@tjanas commented on GitHub (Aug 26, 2022): [Last of Us, The - Part II (USA) (En,Fr,Es,Pt) (Data Disc)_PIC.zip](https://github.com/SabreTools/MPF/files/9435247/Last.of.Us.The.-.Part.II.USA.En.Fr.Es.Pt.Data.Disc._PIC.zip)
claunia commented 2026-01-29 16:16:34 +00:00
Author
Owner
Copy Link

@mnadareski commented on GitHub (Aug 29, 2022):

I tested directly with the PIC and it correctly detected multiple layers from the PIC. So that's telling me that the PIC path, though it should be correct is not.

@mnadareski commented on GitHub (Aug 29, 2022): I tested directly with the PIC and it correctly detected multiple layers from the PIC. So that's telling me that the PIC path, though it _should be correct_ is not.
claunia commented 2026-01-29 16:16:35 +00:00
Author
Owner
Copy Link

@mnadareski commented on GitHub (Sep 28, 2022):

I need someone to test DIC directly. The PIC path that's expected is inside the output folder but if that's changed recently in DIC, then that needs to be accounted for.

@mnadareski commented on GitHub (Sep 28, 2022): I need someone to test DIC directly. The PIC path that's expected is inside the output folder but if that's changed recently in DIC, then that needs to be accounted for.
claunia commented 2026-01-29 16:16:36 +00:00
Author
Owner
Copy Link

@mnadareski commented on GitHub (Sep 28, 2022):

Please try 40bbd422b7, that may have fixed this.

@mnadareski commented on GitHub (Sep 28, 2022): Please try https://github.com/SabreTools/MPF/commit/40bbd422b7835d6c26db132f8b049ed2665a05e7, that may have fixed this.
claunia commented 2026-01-29 16:16:36 +00:00
Author
Owner
Copy Link

@mnadareski commented on GitHub (Sep 29, 2022):

According to two other people who tested this, it is working. Closing this.

@mnadareski commented on GitHub (Sep 29, 2022): According to two other people who tested this, it is working. Closing this.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
duplicate
enhancement
good first issue
help wanted
invalid
pull-request
Mirrored from GitHub Pull Request
 question
regression
wontfix
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SabreTools/MPF#444
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?