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

Selecting Very long path names in the installation does cause icons to be messed up

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Trivial
    • None
    • None
    • None

    Description

      First and foremost. I don't expect anybody to work on this ticket in the coming years. I do have absolutely no usecase for this. I just created the ticket, because it is a failed synthetic test.
      I chose a very long path name in the usetup for ReactOS bootcd to install:

      \ReactOS01234567890ABCDEFGHIJKLMNOPQRSTUVWXYZReact
      

      That is the maximum that I can enter into that edit field, and it is 50 characters long (including the backslash in the beginning).

      It reached the desktop.

      Observed result
      As a consequence many icons in ReactOS were messed up
      0.4.15-dev-6640-g02883d1_icons.png
      0.4.15-dev-6640-g02883d1_icons.log
      the ancient 0.4.7-release-268-g89f60ab was already affected the very same way

      Expected result
      Either not messing up the icons, or limiting the max folder length ros will install to. It is up to you. I would prefer us to not extend fixed-size-buffers sizes endlessly.

      Reminder: If you will fix this ticket, I will retry with an even longer path-name via unattend.inf, and create the next fresh bug if I see anything misbehaving, not to troll you, but because I am acting as a tester here!

      Sidenote: I saw several "punk"-loggings of the form

      aclmulti.cpp:62: Unexpected failure (punk->QueryInterface(IID_PPV_ARG(IACList, &fObjects[fObjectCount].pACL)))=80004002.
      err:(dll/win32/shlwapi/autocomp.cpp:51) punk->QueryInterface failed: 0x80004002
      

      I cannot say, whether they are related or not.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              reactosfanboy reactosfanboy
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: