diff options
| author | Martin Storsjo <martin@martin.st> | 2018-12-20 19:48:39 +0000 | 
|---|---|---|
| committer | Martin Storsjo <martin@martin.st> | 2018-12-20 19:48:39 +0000 | 
| commit | 02e96dd039309dd99a26248ff0934f336d6a188e (patch) | |
| tree | b2777418bd8521a68528f0e191fee536844dbd0b /llvm/tools/llvm-objcopy/COFF/Writer.cpp | |
| parent | f17421e595fe1becbc57fa873ec6278fd7fce188 (diff) | |
| download | bcm5719-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.cpp | 3 | 
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));    }  } | 

