Commit 39e5e45a authored by Andrei Chulkov's avatar Andrei Chulkov Committed by Commit Bot

Update code to avoid casting pointers to long.

Casting pointers to numeric types other than intptr_t is a potential
security issue. Replaced with casts to intptr_t.

Change-Id: I4905401dcc93b89efad8437b0b6f17412cfe2925
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1765710Reviewed-by: default avatarLutz Justen <ljusten@chromium.org>
Reviewed-by: default avatarVasilii Sukhanov <vasilii@chromium.org>
Commit-Queue: Andrei Chulkov <achulkov@google.com>
Cr-Commit-Position: refs/heads/master@{#690416}
parent 56a62c92
...@@ -15,7 +15,7 @@ DownloadLocationDialogBridgeImpl::DownloadLocationDialogBridgeImpl() ...@@ -15,7 +15,7 @@ DownloadLocationDialogBridgeImpl::DownloadLocationDialogBridgeImpl()
: is_dialog_showing_(false) { : is_dialog_showing_(false) {
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
java_obj_.Reset(env, Java_DownloadLocationDialogBridge_create( java_obj_.Reset(env, Java_DownloadLocationDialogBridge_create(
env, reinterpret_cast<long>(this)) env, reinterpret_cast<intptr_t>(this))
.obj()); .obj());
DCHECK(!java_obj_.is_null()); DCHECK(!java_obj_.is_null());
} }
......
...@@ -32,15 +32,15 @@ OverlayWindowAndroid::OverlayWindowAndroid( ...@@ -32,15 +32,15 @@ OverlayWindowAndroid::OverlayWindowAndroid(
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
Java_PictureInPictureActivity_createActivity( Java_PictureInPictureActivity_createActivity(
env, reinterpret_cast<long>(this), env, reinterpret_cast<intptr_t>(this),
TabAndroid::FromWebContents(controller_->GetInitiatorWebContents()) TabAndroid::FromWebContents(controller_->GetInitiatorWebContents())
->GetJavaObject()); ->GetJavaObject());
} }
OverlayWindowAndroid::~OverlayWindowAndroid() { OverlayWindowAndroid::~OverlayWindowAndroid() {
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
Java_PictureInPictureActivity_onWindowDestroyed(env, Java_PictureInPictureActivity_onWindowDestroyed(
reinterpret_cast<long>(this)); env, reinterpret_cast<intptr_t>(this));
} }
void OverlayWindowAndroid::OnActivityStart( void OverlayWindowAndroid::OnActivityStart(
......
...@@ -33,7 +33,7 @@ void OnboardingDialogView::Show() { ...@@ -33,7 +33,7 @@ void OnboardingDialogView::Show() {
ui::WindowAndroid* window_android = ui::WindowAndroid* window_android =
client_->web_contents()->GetTopLevelNativeWindow(); client_->web_contents()->GetTopLevelNativeWindow();
java_object_.Reset(Java_OnboardingDialogBridge_create( java_object_.Reset(Java_OnboardingDialogBridge_create(
env, window_android->GetJavaObject(), reinterpret_cast<long>(this))); env, window_android->GetJavaObject(), reinterpret_cast<intptr_t>(this)));
base::string16 onboarding_title = base::string16 onboarding_title =
l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_ONBOARDING_TITLE); l10n_util::GetStringUTF16(IDS_PASSWORD_MANAGER_ONBOARDING_TITLE);
......
...@@ -29,7 +29,7 @@ PasswordGenerationDialogViewAndroid::PasswordGenerationDialogViewAndroid( ...@@ -29,7 +29,7 @@ PasswordGenerationDialogViewAndroid::PasswordGenerationDialogViewAndroid(
DCHECK(window_android); DCHECK(window_android);
java_object_.Reset(Java_PasswordGenerationDialogBridge_create( java_object_.Reset(Java_PasswordGenerationDialogBridge_create(
base::android::AttachCurrentThread(), window_android->GetJavaObject(), base::android::AttachCurrentThread(), window_android->GetJavaObject(),
reinterpret_cast<long>(this))); reinterpret_cast<intptr_t>(this)));
} }
PasswordGenerationDialogViewAndroid::~PasswordGenerationDialogViewAndroid() { PasswordGenerationDialogViewAndroid::~PasswordGenerationDialogViewAndroid() {
......
...@@ -31,8 +31,9 @@ namespace android { ...@@ -31,8 +31,9 @@ namespace android {
PolicyConverter::PolicyConverter(const Schema* policy_schema) PolicyConverter::PolicyConverter(const Schema* policy_schema)
: policy_schema_(policy_schema), policy_bundle_(new PolicyBundle) { : policy_schema_(policy_schema), policy_bundle_(new PolicyBundle) {
JNIEnv* env = base::android::AttachCurrentThread(); JNIEnv* env = base::android::AttachCurrentThread();
java_obj_.Reset(env, Java_PolicyConverter_create( java_obj_.Reset(
env, reinterpret_cast<long>(this)).obj()); env,
Java_PolicyConverter_create(env, reinterpret_cast<intptr_t>(this)).obj());
DCHECK(!java_obj_.is_null()); DCHECK(!java_obj_.is_null());
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment