SEBWIN-1147: Implemented changes.

Author: dbuechel

SafeExamBrowser.Client/Responsibilities/IntegrityResponsibility.cs

@@ -167,19 +167,9 @@ private void HandleRuntimeIntegrityStatus(bool isValid)
 			{
 				Logger.Info("Runtime integrity successfully verified.");
 			}
-			else if (coordinator.RequestSessionLock())
-			{
-				Logger.Warn("Runtime integrity is compromised!");
-
-				Task.Run(() =>
-				{
-					ShowLockScreen(text.Get(TextKey.LockScreen_RuntimeIntegrityMessage), text.Get(TextKey.LockScreen_Title), Enumerable.Empty<LockScreenOption>());
-					coordinator.ReleaseSessionLock();
-				});
-			}
 			else
 			{
-				Logger.Warn("Runtime integrity is compromised but lock screen is already active!");
+				Logger.Warn("Runtime integrity is compromised!");
 			}
 		}
 

SafeExamBrowser.Configuration/Integrity/IntegrityModule.cs

@@ -194,7 +194,20 @@ public bool TryVerifyRuntimeIntegrity(out bool isValid)
 
 			try
 			{
-				isValid = Native.VerifyRuntimeIntegrity();
+				isValid = Native.VerifyRuntimeIntegrity(out var data, out var count);
+
+				for (var index = 0; index < count; index++)
+				{
+					var pointer = Marshal.ReadIntPtr(data, index * IntPtr.Size);
+					var raw = Marshal.PtrToStringBSTR(pointer);
+					var item = string.Join(" ", raw.ToCharArray().Select(c => Convert.ToInt32(c)));
+
+					logger.Warn($"Runtime Integrity Violation #{index}: {item}");
+
+					Marshal.FreeBSTR(pointer);
+				}
+
+				Marshal.FreeCoTaskMem(data);
 				success = true;
 			}
 			catch (DllNotFoundException)
@@ -326,7 +339,7 @@ private static class Native
 			internal static extern bool VerifyCodeSignature();
 
 			[DllImport(DLL_NAME, CallingConvention = CallingConvention.Cdecl)]
-			internal static extern bool VerifyRuntimeIntegrity();
+			internal static extern bool VerifyRuntimeIntegrity(out IntPtr data, out int count);
 		}
 	}
 }

SafeExamBrowser.Runtime/Operations/Bootstrap/ApplicationIntegrityOperation.cs

@@ -29,19 +29,13 @@ public ApplicationIntegrityOperation(IIntegrityModule module, ILogger logger)
 
 		public OperationResult Perform()
 		{
-			var result = OperationResult.Success;
-
-			logger.Info($"Attempting to verify application integrity...");
+			logger.Info("Attempting to verify application and runtime integrity...");
 			StatusChanged?.Invoke(TextKey.OperationStatus_VerifyApplicationIntegrity);
 
 			VerifyCodeSignature();
+			VerifyRuntimeIntegrity();
 
-			if (!VerifyRuntimeIntegrity())
-			{
-				result = OperationResult.Failed;
-			}
-
-			return result;
+			return OperationResult.Success;
 		}
 
 		public OperationResult Revert()
@@ -78,7 +72,7 @@ private bool VerifyRuntimeIntegrity()
 				}
 				else
 				{
-					logger.Error("Runtime integrity is compromised!");
+					logger.Warn("Runtime integrity is compromised!");
 				}
 			}
 			else

SafeExamBrowser.Runtime/Responsibilities/IntegrityResponsibility.cs

@@ -7,7 +7,6 @@
  */
 
 using System;
-using System.Threading.Tasks;
 using System.Timers;
 using SafeExamBrowser.Configuration.Contracts.Integrity;
 using SafeExamBrowser.Logging.Contracts;
@@ -88,19 +87,7 @@ private void HandleRuntimeIntegrityStatus(bool isValid)
 			}
 			else
 			{
-				Logger.Error("Runtime integrity is compromised!");
-
-				StopIntegrityMonitoring();
-
-				Task.Run(() =>
-				{
-					if (SessionIsRunning)
-					{
-						Context.Responsibilities.Delegate(RuntimeTask.StopSession);
-					}
-
-					shutdown.Invoke();
-				});
+				Logger.Warn("Runtime integrity is compromised!");
 			}
 		}
 	}