Skip to content

File and directory collision bug #17

Open

Description

@shidel

Reported on SourceForge as Issue 365

According to the bug report... Basically, LABEL is overwriting root directory entries for files and directories without checking for conflicts. Without testing myself, I would assume it is probably also corrupting the filesystem when it overwrites an entry.

Activity

  1. andrewbird commented on Aug 14, 2022

    @andrewbird
    Contributor

    I've written some tests (calls the int21/16 function directly) for Dosemu2's test suite that show that at least some part of the problem is in the kernel.

    FreeDOS (git and 1.2) kernel on FAT does:
    a) With existing file, volume is set but the existing file is deleted
    b) With existing directory, the function returns failure.

    With MS-DOS 6.22 and DR-DOS 7.01 the existing file or directory is left intact and the volume label is set in a duplicate file, but with the VOL attr set(effectively hidden). These DOSs also set the volume name in the BPB.

    Edit: updated some incorrect statements.

  2. bttrx commented on Aug 19, 2022

    @bttrx

    Probably connected or the same issue: https://gitlab.com/FreeDOS/issue-reporting/-/issues/31

  3. andrewbird commented on Aug 19, 2022

    @andrewbird
    Contributor

    Thanks Robert, I have seen this already and I do agree that it's probably connected. Once I figure out what exactly is required in both Label and Kernel I'll try to submit patches.
    Thanks again for the heads-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions