RFR: 8217216: Launcher does not defend itself against LD_LIBRARY_PATH_64 (Solaris)

[Roger Riggs]

Hi Henry,

The change looks ok, Reviewed.

(I'm not that familiar with Solaris).

Thanks, Roger



On 3/1/19 10:45 AM, Henry Jen wrote:
> Ping!
>
> Any thought on this webrev restore the way for Solaris LD_LIBRARY_PATH_64 handling?
>
> Cheers,
> Henry
>
>
>> On Feb 13, 2019, at 9:37 AM, Henry Jen <henry.jen at oracle.com> wrote:
>>
>> Hi,
>>
>> Please review the webrev[1] for 8217216. The fix makes sure on Solaris, when LD_LIBRARY_PATH_64 is set, we setup LD_LIBRARY_PATH based on that value and unset LD_LIBRARY_PATH_64 in the relaunched process.
>>
>> Same approach was used before JDK-6367077, and the override is expected behavior on Solaris 64-bit as stated in Solaris 64-bit Developer's Guide[2],
>> "The 64-bit dynamic linker's search path can be completely overridden using the LD_LIBRARY_PATH_64 environment variable."
>>
>> Cheers,
>> Henry
>>
>> [1] http://cr.openjdk.java.net/~henryjen/8217216/0/webrev/
>> [2] https://docs.oracle.com/cd/E19455-01/806-0477/dev-env-7/index.html
>>