summaryrefslogtreecommitdiffstats
path: root/llvm/tools/llvm-objcopy/COFF/Writer.cpp
diff options
context:
space:
mode:
authorMartin Storsjo <martin@martin.st>2018-12-20 19:48:39 +0000
committerMartin Storsjo <martin@martin.st>2018-12-20 19:48:39 +0000
commit02e96dd039309dd99a26248ff0934f336d6a188e (patch)
treeb2777418bd8521a68528f0e191fee536844dbd0b /llvm/tools/llvm-objcopy/COFF/Writer.cpp
parentf17421e595fe1becbc57fa873ec6278fd7fce188 (diff)
downloadbcm5719-llvm-02e96dd039309dd99a26248ff0934f336d6a188e.tar.gz
bcm5719-llvm-02e96dd039309dd99a26248ff0934f336d6a188e.zip
[llvm-objcopy] [COFF] Don't call memcpy() with a null argument. NFC.
It is invalid to call memcpy with a null pointer, even if the size is zero. This should fix the sanitizer buildbot. llvm-svn: 349808
Diffstat (limited to 'llvm/tools/llvm-objcopy/COFF/Writer.cpp')
-rw-r--r--llvm/tools/llvm-objcopy/COFF/Writer.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/tools/llvm-objcopy/COFF/Writer.cpp b/llvm/tools/llvm-objcopy/COFF/Writer.cpp
index 48afe7889e8..6cda11c8a6b 100644
--- a/llvm/tools/llvm-objcopy/COFF/Writer.cpp
+++ b/llvm/tools/llvm-objcopy/COFF/Writer.cpp
@@ -225,7 +225,8 @@ void COFFWriter::writeSections() {
S.Header.SizeOfRawData - S.Contents.size());
Ptr += S.Header.SizeOfRawData;
- memcpy(Ptr, S.Relocs.data(), S.Relocs.size() * sizeof(coff_relocation));
+ if (!S.Relocs.empty())
+ memcpy(Ptr, S.Relocs.data(), S.Relocs.size() * sizeof(coff_relocation));
}
}
OpenPOWER on IntegriCloud