Fix conversions from 32-bit pointers to signed 64-bit integers.

Fix conversions from 32-bit pointers to signed 64-bit integers.

In case the address is a negative 32-bit integer (greater than 0x80000000), it
is converted into 64-bit preserving the sign so all the leading bits are set.
This results in an invalid 64-bit address.

The conversion must be done via an unsigned 32-bit integer (uintptr_t).

BUG=409826
Committed: https://crrev.com/1ddc8eb883d03a3dac175832357e5b2889e15134
Cr-Commit-Position: refs/heads/master@{#293140}

[whywhat, 2014-09-03 14:12:30]

avayvod@chromium.org changed reviewers:
+ nyquist@chromium.org, simonb@chromium.org, yfriedman@chromium.org

[Yaron, 2014-09-03 14:24:56]

lgtm

https://codereview.chromium.org/533173002/diff/1/base/android/linker/linker_j...
File base/android/linker/linker_jni.cc (right):

https://codereview.chromium.org/533173002/diff/1/base/android/linker/linker_j...
base/android/linker/linker_jni.cc:26: #define DEBUG 1
revert

[whywhat, 2014-09-03 14:40:28]

https://codereview.chromium.org/533173002/diff/1/base/android/linker/linker_j...
File base/android/linker/linker_jni.cc (right):

https://codereview.chromium.org/533173002/diff/1/base/android/linker/linker_j...
base/android/linker/linker_jni.cc:26: #define DEBUG 1
On 2014/09/03 14:24:56, Yaron wrote:
> revert

Done.

[whywhat, 2014-09-03 14:40:53]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2014-09-03 14:41:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/avayvod@chromium.org/533173002/20001

[simonb (inactive), 2014-09-03 15:09:29]

lgtm

Thanks for fixing.

[commit-bot: I haz the power, 2014-09-03 15:41:51]

Committed patchset #2 (id:20001) as ceb0b2d38c6de666735519dd54a34530742398b8

[commit-bot: I haz the power, 2014-09-10 03:25:52]

Patchset 2 (id:??) landed as
https://crrev.com/1ddc8eb883d03a3dac175832357e5b2889e15134
Cr-Commit-Position: refs/heads/master@{#293140}