Move close() to the proper else block

[yingcong-wu]

Summary

LogWriter::Close(LW) is outside the null check if-else block, which, when LW == nullptr, will causing a NULL dereference.
I think the close() means to be in else block, which is when LW != nullptr.

[xgupta]

LGTM, Thanks.
Seems reasonable.

compiler-rt/lib/xray/xray_profiling.cpp

if (B.Data == nullptr) {
       } else {
         // Now for each of the buffers, write out the profile data as we would
         // see it in memory, verbatim.
         while (B.Data != nullptr && B.Size != 0) {
           LW->WriteAll(reinterpret_cast<const char *>(B.Data),
                        reinterpret_cast<const char *>(B.Data) + B.Size);
           B = profileCollectorService::nextBuffer(B);
         }
-      }
         LogWriter::Close(LW);
       }
     }
+  }
 
   profileCollectorService::reset();
 
   atomic_store(&ProfilerLogFlushStatus, XRayLogFlushStatus::XRAY_LOG_FLUSHED,
                memory_order_release);
   atomic_store(&ProfilerLogStatus, XRayLogInitStatus::XRAY_LOG_UNINITIALIZED,
                memory_order_release);