[sql] WebSQL xGetLastError() can be called with null buffer.

third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp

Index: third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp
diff --git a/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp b/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp
index 21bacd7f5e0ff032227ba7d952e974a22fbea703..d59502eeeb646c9086f653f611a6f313eb27a64b 100644
--- a/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp
+++ b/third_party/WebKit/Source/modules/webdatabase/sqlite/SQLiteFileSystemPosix.cpp
@@ -328,10 +328,9 @@ int chromiumCurrentTime(sqlite3_vfs* vfs, double* prNow) {
   return wrappedVfs->xCurrentTime(wrappedVfs, prNow);
 }
 
-int chromiumGetLastError(sqlite3_vfs* vfs, int e, char* s) {
-  // xGetLastError() has never been used by SQLite.  The implementation in
-  // os_win.c indicates this is a reasonable implementation.
-  *s = '\0';
+int chromiumGetLastError(sqlite3_vfs* vfs, int nBuf, char* zBuf) {

[michaeln, 2017/03/23 20:36:33]

lgtm

+  if (nBuf && zBuf)
+    *zBuf = '\0';
   return 0;
 }