microsoft / WindowsAppSDK-Samples

Feature samples for the Windows App SDK
MIT License
721 stars 225 forks source link

Fix Prefast C6387 on SimpleIslandApp.cpp #332

Closed kythant closed 5 months ago

kythant commented 5 months ago

Prefast results shows

hWnd' could be '0': this does not adhere to the specification for the function 'ShowWindow hWnd' could be '0': this does not adhere to the specification for the function 'UpdateWindow'. See line 199 for an earlier location where this can occur

The fix was to add a null check and throw if null

kythant commented 5 months ago

/azp run Samples-Islands

azure-pipelines[bot] commented 5 months ago
No pipelines are associated with this pull request.
kythant commented 5 months ago

/azp run SamplesCI-Islands

azure-pipelines[bot] commented 5 months ago
Azure Pipelines successfully started running 1 pipeline(s).
kythant commented 5 months ago

/azp run SamplesCI-Islands

azure-pipelines[bot] commented 5 months ago
Azure Pipelines successfully started running 1 pipeline(s).