Title: [170510] trunk/Tools
Revision
170510
Author
[email protected]
Date
2014-06-26 16:59:10 -0700 (Thu, 26 Jun 2014)

Log Message

[Win] Fix potential buffer overrun in DLLLauncher
https://bugs.webkit.org/show_bug.cgi?id=134351
<rdar://problem/17469185>

Reviewed by Darin Adler.

* win/DLLLauncher/DLLLauncherMain.cpp: Correct error
in end-of-buffer calculation.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (170509 => 170510)


--- trunk/Tools/ChangeLog	2014-06-26 23:54:11 UTC (rev 170509)
+++ trunk/Tools/ChangeLog	2014-06-26 23:59:10 UTC (rev 170510)
@@ -1,3 +1,14 @@
+2014-06-26  Brent Fulgham  <[email protected]>
+
+        [Win] Fix potential buffer overrun in DLLLauncher
+        https://bugs.webkit.org/show_bug.cgi?id=134351
+        <rdar://problem/17469185>
+
+        Reviewed by Darin Adler.
+
+        * win/DLLLauncher/DLLLauncherMain.cpp: Correct error
+        in end-of-buffer calculation.
+
 2014-06-26  Tamas Gergely  <[email protected]>
 
         [EFL] Missing dependency between freetype6 and fontconfig libs.

Modified: trunk/Tools/win/DLLLauncher/DLLLauncherMain.cpp (170509 => 170510)


--- trunk/Tools/win/DLLLauncher/DLLLauncherMain.cpp	2014-06-26 23:54:11 UTC (rev 170509)
+++ trunk/Tools/win/DLLLauncher/DLLLauncherMain.cpp	2014-06-26 23:59:10 UTC (rev 170510)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012-2013 Apple Inc. All rights reserved.
+ * Copyright (C) 2012-2014 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -176,7 +176,7 @@
     if (len >= bufSize)
         len = bufSize - 1;
 
-    errorMessage[len + 1] = 0;
+    errorMessage[len] = 0;
 
     return errorMessage;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to