fix: sef fault on linux due to non existent path (#51)

awslabs · Oct 25, 2021 · 4e670ac · 4e670ac
1 parent 081b0f9
commit 4e670ac
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/Source/AwsCommonRuntimeKit/sdkutils/CRTAWSProfileCollection.swift b/Source/AwsCommonRuntimeKit/sdkutils/CRTAWSProfileCollection.swift
@@ -6,19 +6,22 @@ import AwsCSdkUtils
 public class CRTAWSProfileCollection {
     var rawValue: OpaquePointer
 
-    public init(fromFile path: String,
-                source: CRTAWSProfileSourceType,
-                allocator: Allocator = defaultAllocator) {
+    public init?(fromFile path: String,
+                 source: CRTAWSProfileSourceType,
+                 allocator: Allocator = defaultAllocator) {
         var finalizedPath = path
         if path.hasPrefix("~"),
            let homeDirectory = aws_get_home_directory(allocator.rawValue),
            let homeDirectoryString = String(awsString: homeDirectory) {
             finalizedPath = homeDirectoryString + path.dropFirst()
         }
         let awsString = AWSString(finalizedPath, allocator: allocator)
-        self.rawValue = aws_profile_collection_new_from_file(allocator.rawValue,
-                                                             awsString.rawValue,
-                                                             source.rawValue)
+        guard let profilePointer = aws_profile_collection_new_from_file(allocator.rawValue,
+                                                                        awsString.rawValue,
+                                                                        source.rawValue) else {
+            return nil
+        }
+        self.rawValue = profilePointer
     }
 
     public init(fromBuffer buffer: ByteBuffer,
@@ -28,9 +31,9 @@ public class CRTAWSProfileCollection {
         let byteCount = byteArray.count
         let byteBuf = byteArray.withUnsafeMutableBufferPointer { pointer -> aws_byte_buf in
             let byteBuf = aws_byte_buf(len: byteCount,
-                                buffer: pointer.baseAddress,
-                                capacity: byteCount,
-                                allocator: allocator.rawValue)
+                                       buffer: pointer.baseAddress,
+                                       capacity: byteCount,
+                                       allocator: allocator.rawValue)
             return byteBuf
         }
         let pointer: UnsafePointer<aws_byte_buf> = fromPointer(ptr: byteBuf)