fix: address static analysis warnings in SQLite and WebStorage

Author: GoldFish2500

src/node_sqlite.cc

@@ -97,7 +97,11 @@ using v8::Value;
       case SQLITE_TEXT: {                                                      \
         const char* v =                                                        \
             reinterpret_cast<const char*>(sqlite3_##from##_text(__VA_ARGS__)); \
-        (result) = String::NewFromUtf8((isolate), v).As<Value>();              \
+        if (v == nullptr) {                                                    \
+            (result) = Null((isolate));                                        \
+        } else {                                                               \
+            (result) = String::NewFromUtf8((isolate), v).As<Value>();          \
+        }                                                                      \
         break;                                                                 \
       }                                                                        \
       case SQLITE_NULL: {                                                      \
@@ -246,7 +250,7 @@ inline void THROW_ERR_SQLITE_ERROR(Isolate* isolate, int errcode) {
 
   Environment* env = Environment::GetCurrent(isolate);
   Local<Object> error;
-  if (CreateSQLiteError(isolate, errstr).ToLocal(&error) &&
+  if (env && CreateSQLiteError(isolate, errstr).ToLocal(&error) &&
       error
           ->Set(isolate->GetCurrentContext(),
                 env->errcode_string(),

src/node_webstorage.cc

@@ -177,6 +177,7 @@ Maybe<void> Storage::Open() {
   sqlite3_stmt* s = nullptr;
   r = sqlite3_prepare_v2(
       db, get_schema_version_sql.data(), get_schema_version_sql.size(), &s, 0);
+  CHECK_ERROR_OR_THROW(env(), r, SQLITE_OK, Nothing<void>());
   r = sqlite3_exec(db, init_sql_v0.data(), 0, 0, nullptr);
   CHECK_ERROR_OR_THROW(env(), r, SQLITE_OK, Nothing<void>());
   auto stmt = stmt_unique_ptr(s);