Android: add better nullability checks for nullability annotations added in NDK 26

Author: finagolfin

Sources/TSCBasic/FileSystem.swift

@@ -491,8 +491,8 @@ private struct LocalFileSystem: FileSystem {
 
     func readFileContents(_ path: AbsolutePath) throws -> ByteString {
         // Open the file.
-        let fp = fopen(path.pathString, "rb")
-        if fp == nil {
+        let fpo = fopen(path.pathString, "rb")
+        guard let fp = fpo else {
             throw FileSystemError(errno: errno, path)
         }
         defer { fclose(fp) }
@@ -521,8 +521,8 @@ private struct LocalFileSystem: FileSystem {
 
     func writeFileContents(_ path: AbsolutePath, bytes: ByteString) throws {
         // Open the file.
-        let fp = fopen(path.pathString, "wb")
-        if fp == nil {
+        let fpo = fopen(path.pathString, "wb")
+        guard let fp = fpo else {
             throw FileSystemError(errno: errno, path)
         }
         defer { fclose(fp) }

Sources/TSCBasic/Process.swift

@@ -147,6 +147,9 @@ public final class Process {
 
         /// The current OS does not support the workingDirectory API.
         case workingDirectoryNotSupported
+
+        /// The stdin could not be opened.
+        case stdinNotOpening
     }
 
     public enum OutputRedirection {
@@ -680,7 +683,10 @@ public final class Process {
         var stdinPipe: [Int32] = [-1, -1]
         try open(pipe: &stdinPipe)
 
-        let stdinStream = try LocalFileOutputByteStream(filePointer: fdopen(stdinPipe[1], "wb"), closeOnDeinit: true)
+        guard let fp = fdopen(stdinPipe[1], "wb") else {
+            throw Process.Error.stdinNotOpening
+        }
+        let stdinStream = try LocalFileOutputByteStream(filePointer: fp, closeOnDeinit: true)
 
         // Dupe the read portion of the remote to 0.
         posix_spawn_file_actions_adddup2(&fileActions, stdinPipe[0], 0)
@@ -1265,6 +1271,8 @@ extension Process.Error: CustomStringConvertible {
             return "could not find executable for '\(program)'"
         case .workingDirectoryNotSupported:
             return "workingDirectory is not supported in this platform"
+        case .stdinNotOpening:
+            return "could not open stdin on this platform"
         }
     }
 }

Sources/TSCBasic/WritableByteStream.swift

@@ -790,7 +790,11 @@ public final class LocalFileOutputByteStream: FileOutputByteStream {
     override final func writeImpl(_ bytes: ArraySlice<UInt8>) {
         bytes.withUnsafeBytes { bytesPtr in
             while true {
+              #if os(Android)
+                let n = fwrite(bytesPtr.baseAddress!, 1, bytesPtr.count, filePointer)
+              #else
                 let n = fwrite(bytesPtr.baseAddress, 1, bytesPtr.count, filePointer)
+              #endif
                 if n < 0 {
                     if errno == EINTR { continue }
                     errorDetected(code: errno)

Sources/TSCTestSupport/PseudoTerminal.swift

@@ -24,7 +24,7 @@ public final class PseudoTerminal {
         if openpty(&primary, &secondary, nil, nil, nil) != 0 {
             return nil
         }
-        guard let outStream = try? LocalFileOutputByteStream(filePointer: fdopen(secondary, "w"), closeOnDeinit: false) else {
+        guard let outStream = try? LocalFileOutputByteStream(filePointer: fdopen(secondary, "w")!, closeOnDeinit: false) else {
             return nil
         }
         self.outStream = outStream