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

After skipped IopVerifyDiskSignature() call, 'Signature' is used as an uninitialized variable

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • NTCore

    Description

      (with reference to revision 73546)
      (sanity/auditing functionality of probable low priority)
      (Apparently the compiler(s) are not catching this one...)
      (appears only applicable to startup process, though I don't think this is actually 'bootloader' code perse though think only accessed at boot time)
      (not actually observed to happen, only from from examination while searching for other issue, but think this holds true)

      in ntoskrnl/io/iomgr/arcname.c:r73546, involving code in lines 702:803, questionable code flow exists from if condition around line 702 to else-if around line 797.

      Appears could have 'truth' of (this partial) condition in 'if()'
      // (((SingleDisk && DiskCount == 1)
      // && !(DriveLayout->PartitionStyle == PARTITION_STYLE_MBR))
      (note I added the '!' for reading of situation leading to line 797) in which case the 'Signature' value would NOT have been init'd by a call to IopVerifyDiskSignature() for the particular disk/partition being checked, and thus the audit check itself would be using an unknown/unrelated value, possibly either failing to report or incorrectly reporting, depending on circumstances of particular instance.

      I have no idea whether or not such a set of circumstances actually exists in the real world.

      Attachments

        Activity

          People

            bug zilla Bug Zilla
            curiousone curiousone
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: