Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rundll32: fix documentation and add more detail #8026

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nir9
Copy link

@nir9 nir9 commented Feb 6, 2025

Problem: Current rundll32 documentation contains incorrect information and lacks information about the syntax.

Example: Currently, the doc claims that rundll32 loads only 32bit DLLs which is untrue, I have tested with a 64bit Windows 10 machine and when rundll32 is ran regularly is loads 64bit DLLs and when it is ran through SysWow64 like so for example C:\Windows\SysWow64\rundll32 kernel32,Beep it of course uses 32bit DLLs

Here is a screenshot from procexp that shows it uses the 64bit DLL:
image

Solution:

  • Remove 32bit only claim
  • Remove claim that it requires admin rights (not needed necessarily)
  • Add full syntax information and additional remarks

Copy link
Contributor

@nir9 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 79ab8c0:

⚠️ Validation status: warnings

File Status Preview URL Details
WindowsServerDocs/administration/windows-commands/rundll32.md ⚠️Warning Details

WindowsServerDocs/administration/windows-commands/rundll32.md

  • Line 38, Column 69: [Warning: file-not-found - See documentation] Invalid file link: 'windows/win32/learnwin32/winmain--the-application-entry-point'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@v-dirichards
Copy link
Contributor

@robinharwood, @Xelu86
Can you review the proposed changes?

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team
#assign: @robinharwood, @Xelu86

Copy link
Contributor

Users robinharwood are already assigned.

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Feb 6, 2025
Copy link
Contributor

Learn Build status updates of commit 5fe4345:

✅ Validation status: passed

File Status Preview URL Details
WindowsServerDocs/administration/windows-commands/rundll32.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

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

Successfully merging this pull request may close these issues.

None yet

4 participants