From a2c6ae6f841d088b79a2a7bbd73847b8b199a521 Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Wed, 27 Aug 2014 13:45:37 -0700 Subject: [PATCH] Add missing type casts before comparison. BUG: 17300548 Change-Id: Ice9868f36c8fa8cd40bb13741b0e33c8f8d354fd --- tests/dlext_test.cpp | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/dlext_test.cpp b/tests/dlext_test.cpp index da630463d..9a96b67d0 100644 --- a/tests/dlext_test.cpp +++ b/tests/dlext_test.cpp @@ -117,7 +117,7 @@ TEST_F(DlExtTest, Reserved) { ASSERT_DL_NOTNULL(handle_); fn f = reinterpret_cast(dlsym(handle_, "getRandomNumber")); ASSERT_DL_NOTNULL(f); - EXPECT_GE(f, start); + EXPECT_GE(reinterpret_cast(f), start); EXPECT_LT(reinterpret_cast(f), reinterpret_cast(start) + LIBSIZE); EXPECT_EQ(4, f()); @@ -147,7 +147,7 @@ TEST_F(DlExtTest, ReservedHint) { ASSERT_DL_NOTNULL(handle_); fn f = reinterpret_cast(dlsym(handle_, "getRandomNumber")); ASSERT_DL_NOTNULL(f); - EXPECT_GE(f, start); + EXPECT_GE(reinterpret_cast(f), start); EXPECT_LT(reinterpret_cast(f), reinterpret_cast(start) + LIBSIZE); EXPECT_EQ(4, f()); @@ -165,8 +165,9 @@ TEST_F(DlExtTest, ReservedHintTooSmall) { ASSERT_DL_NOTNULL(handle_); fn f = reinterpret_cast(dlsym(handle_, "getRandomNumber")); ASSERT_DL_NOTNULL(f); - EXPECT_TRUE(f < start || (reinterpret_cast(f) >= - reinterpret_cast(start) + PAGE_SIZE)); + EXPECT_TRUE(reinterpret_cast(f) < start || + (reinterpret_cast(f) >= + reinterpret_cast(start) + PAGE_SIZE)); EXPECT_EQ(4, f()); }