Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-13226

base/setup/usetup/bootsup.c: "C:\\bootsect.ros" should not assume "C:"

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Fix
    • Minor
    • None
    • Setup
    • None

    Description

      (A step to fixing CORE-13205: see 2_9.png.)

      I didn't look too close, but it seems line 2363 should use something like "SystemRootPath->Buffer" too.

      bootsect.ros grep

      reactos/base/setup/usetup/bootsup.c
      2322             wcscat(DstPath, L"\\bootsect.ros");
      2363                                L"C:\\bootsect.ros",
       
      Both are in
      2249 InstallFatBootcodeToPartition(

      I think there might be 3 cases:
      *ROS on C: Trivial, no problem.
      *ROS on D: Minor, works though file on wrong partition.
      *ROS on C and D: Major, may overwrite/share (previous) file.

      Attachments

        Issue Links

          Activity

            People

              hbelusca hbelusca
              Serge Gautherie Serge Gautherie
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: