dMZX Forums: Emscripten: save corruption with frontend, limited filesystem operations -> Archived MegaZeux Bugs -> Tracker

Jump to content

Report ID 789 Title Emscripten: save corruption with frontend, limited filesystem operations
Product Archived MegaZeux Bugs Status Fixed (Severity 2 - Fair)
Version 2.92b Fixed in 2.92b
Introduced In Version 2.92bOperating System HTML5

Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Report ID #789: Emscripten: save corruption with frontend, limited filesystem operations

#1 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,904
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 21 September 2019 - 11:14 PM

The frontend for Emscripten builds fails to implement the setattr size field. This field is used by the Emscripten FS API to perform O_TRUNC when opening files as "w" or "w+". Lack of support for this causes corruption when overwriting saves or other files, as remnant data from the previous file persists at the end of the new file.

The following operations are not implemented at all, meaning there's no easy way to rename or remove specific files (including but not limited to the aforementioned corrupt files):
  • rename
  • unlink
  • rmdir

"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository


Page 1 of 1  
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Replies (1 - 3)

#2 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,904
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 02 October 2019 - 12:33 AM

Updating version to: 2.92b
Updating Introduced In Version to: 2.92b

unlink and rmdir were fixed by the changes in GIT 0552ae29. rename is still pending (see this comment on the PR for that commit for more info, but basically, the way the FS API works and the way the frontend has to work make this difficult).
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#3 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,904
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 29 April 2020 - 03:46 AM

Updating status to: Fixed
Issue fixed in: 2.92b

All of this was fixed in the above commit except for rename(), which I've opened a new issue for. Closing this.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#4 User is offline  
Terryn 

  • ******
  • Group: DigiStaff
  • Posts: 2,961
  • Joined: 12-October 00
  • Gender:Male

Posted 09 May 2020 - 12:12 AM

Moving to: Archived MegaZeux Bugs


Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

1 User(s) are reading this issue
1 Guests and 0 Anonymous Users


Powered by IP.Tracker 1.3.2 © 2024  IPS, Inc.