src,build: adjust suppressions file and ub on src

Co-Authored-By: Santiago Gimeno <[email protected]>

src/spawn_sync.cc

@@ -68,7 +68,8 @@ void SyncProcessOutputBuffer::OnRead(const uv_buf_t* buf, size_t nread) {
 
 
 size_t SyncProcessOutputBuffer::Copy(char* dest) const {
-  memcpy(dest, data_, used());
+  if (dest != nullptr)
+    memcpy(dest, data_, used());
   return used();
 }
 

suppressions.supp

@@ -1,2 +1,6 @@
-undefined-behavior:deps/icu-small/source/common/uloc_tag.cpp
-undefined-behavior:deps/v8/*
+vptr:deps/icu-small/source/common/uloc_tag.cpp
+vptr:deps/icu-small/source/common/unistr.cpp
+shift-base:deps/v8/src/wasm/decoder.h
+vptr:deps/icu-small/source/common/sharedobject.cpp
+vptr:deps/icu-small/source/i18n/coll.cpp
+nonnull-attribute:deps/v8/src/snapshot/snapshot-source-sink.h

test/parallel/test-node-output-console.mjs

@@ -31,7 +31,11 @@ describe('console output', { concurrency: true }, () => {
     .transform(snapshot.replaceWindowsLineEndings, snapshot.replaceWindowsPaths, replaceStackTrace);
   for (const { name, transform, env } of tests) {
     it(name, async () => {
-      await snapshot.spawnAndAssert(fixtures.path(name), transform ?? defaultTransform, { env });
+      await snapshot.spawnAndAssert(
+        fixtures.path(name),
+        transform ?? defaultTransform,
+        { env: { ...env, ...process.env } },
+      );
     });
   }
 });