[PATCH] D139449: [ObjYAML] Pass 10 instead of 16 for strncmp

Alf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 12:00:44 PST 2022


gAlfonso-bit updated this revision to Diff 480569.
gAlfonso-bit added a comment.

Use strcmp instead of strncmp


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139449/new/

https://reviews.llvm.org/D139449

Files:
  llvm/lib/ObjectYAML/MachOEmitter.cpp


Index: llvm/lib/ObjectYAML/MachOEmitter.cpp
===================================================================
--- llvm/lib/ObjectYAML/MachOEmitter.cpp
+++ llvm/lib/ObjectYAML/MachOEmitter.cpp
@@ -301,7 +301,7 @@
       uint64_t segOff = is64Bit ? LC.Data.segment_command_64_data.fileoff
                                 : LC.Data.segment_command_data.fileoff;
       if (0 ==
-          strncmp(&LC.Data.segment_command_data.segname[0], "__LINKEDIT", 16)) {
+          strcmp(&LC.Data.segment_command_data.segname[0], "__LINKEDIT")) {
         FoundLinkEditSeg = true;
         LinkEditOff = segOff;
         if (Obj.RawLinkEditSegment)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D139449.480569.patch
Type: text/x-patch
Size: 639 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221206/d41eaef1/attachment.bin>


More information about the llvm-commits mailing list