microsoft / microsoft/win32metadata

Incorrect setup for `WTSEnumerateProcesses()`.

Open
#2,144 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
C++
Stars
1.5k
Forks
149
Avg merge
5d 16h
Merged PRs (30d)
4

Description

## Actual behavior

The `pCount` parameter for this method is by `ref`, but it should be an `out` parameter just like it is for `WTSEnumerateSessions()` and the [documentation](https://learn.microsoft.com/en-us/windows/win32/api/wtsapi32/nf-wtsapi32-wtsenumerateprocessesw) says.

## Expected behavior

That the `pCount` parameter for `WTSEnumerateProcesses()` be an `out` parameter, not a `ref` parameter.

## Repro steps

1. `NativeMethods.txt` content:
```
WTSEnumerateProcessesW
```

2. `NativeMethods.json` content (if present):
N/A

3. Any of your own code that should be shared?

### Context

- CsWin32 version: 0.3.183
- Win32Metadata version (if explicitly set by project): N/A
- Target Framework: net472
- `LangVersion` (if explicitly set by project): N/A

Contributor guide

Open the contributing guide

Research direction

Start by locating the metadata entry for WTSEnumerateProcessesW and compare its pCount parameter with WTSEnumerateSessions and the linked Windows documentation. Reproduce the generated signature using the provided NativeMethods.txt input, then verify that pCount is emitted as an out parameter rather than ref.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
devtools
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.