Skip to content

Commit 81ea3e4

Browse files
BinExport Authorscopybara-github
authored andcommitted
Make architecture names case insensitive.
Architecture names do not always come lower case. For example, IDA plugin does not do that: https://github.com/google/binexport/blob/a8572981126e455b99ba813edbc9a88a3269d8b6/ida/names.cc#L101 PiperOrigin-RevId: 691769928 Change-Id: I3014e85201cb6d66c7f8f17210bfc05f419100b7
1 parent 3a9eeea commit 81ea3e4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

reader/flow_graph.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ namespace security::binexport {
3030
namespace {
3131

3232
absl::optional<Architecture> GetSupportedArchitecture(const BinExport2& proto) {
33-
const std::string& architecture =
34-
proto.meta_information().architecture_name();
33+
const std::string architecture =
34+
absl::AsciiStrToLower(proto.meta_information().architecture_name());
3535
if (architecture == "arm") {
3636
return Architecture::kArm;
3737
}

0 commit comments

Comments
 (0)