From 36740ab4a21a1cd51f8cd795f295a86aa5c91d34 Mon Sep 17 00:00:00 2001 From: Patrick Meinecke Date: Mon, 9 Dec 2024 16:25:29 -0500 Subject: [PATCH] Assign the value returned by the `MaybeAdd` method --- .../engine/runtime/Binding/Binders.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/System.Management.Automation/engine/runtime/Binding/Binders.cs b/src/System.Management.Automation/engine/runtime/Binding/Binders.cs index ab4e1067ac..78d459933f 100644 --- a/src/System.Management.Automation/engine/runtime/Binding/Binders.cs +++ b/src/System.Management.Automation/engine/runtime/Binding/Binders.cs @@ -6947,7 +6947,7 @@ namespace System.Management.Automation.Language // - Log method invocation to AMSI Notifications (can throw PSSecurityException) // - Invoke method string targetName = methodInfo.ReflectedType?.FullName ?? string.Empty; - MaybeAddMemberInvocationLogging(expr, targetName, name, args); + expr = MaybeAddMemberInvocationLogging(expr, targetName, name, args); // If we're calling SteppablePipeline.{Begin|Process|End}, we don't want // to wrap exceptions - this is very much a special case to help error