test: skip FFI reentrant callback test on SmartOS

Author: trivikr

lib/internal/ffi/fast-api.js

@@ -297,7 +297,7 @@ function wrapWithRawPointerConversions(rawFn, argumentTypes, _owner) {
       try {
         return rawFn(c0 ?
           convertFastArg(t0, a0, rawFn, stringState, 0) : a0,
-        c1 ? convertFastArg(t1, a1, rawFn, stringState, 1) : a1);
+                     c1 ? convertFastArg(t1, a1, rawFn, stringState, 1) : a1);
       } finally {
         if (stringCall) exitStringConversion(stringState);
       }
@@ -320,8 +320,8 @@ function wrapWithRawPointerConversions(rawFn, argumentTypes, _owner) {
       try {
         return rawFn(c0 ?
           convertFastArg(t0, a0, rawFn, stringState, 0) : a0,
-        c1 ? convertFastArg(t1, a1, rawFn, stringState, 1) : a1,
-        c2 ? convertFastArg(t2, a2, rawFn, stringState, 2) : a2);
+                     c1 ? convertFastArg(t1, a1, rawFn, stringState, 1) : a1,
+                     c2 ? convertFastArg(t2, a2, rawFn, stringState, 2) : a2);
       } finally {
         if (stringCall) exitStringConversion(stringState);
       }

test/ffi/test-ffi-fast-buffer.js

@@ -70,7 +70,10 @@ test('fast FFI buffer arguments reject invalid values', () => {
   }
 });
 
-test('fast FFI string buffers survive reentrant callbacks', () => {
+test('fast FFI string buffers survive reentrant callbacks', {
+  // Bundled libffi callbacks crash on SmartOS.
+  skip: common.isSunOS,
+}, () => {
   const { lib, functions } = ffi.dlopen(libraryPath, {
     safe_strlen: { arguments: ['string'], return: 'i32' },
     string_survives_callback: {