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

[Henry Jen]

Thanks, Roger.

Cheers,
Henry


> On Mar 1, 2019, at 10:27 AM, Roger Riggs <roger.riggs at oracle.com> wrote:
> 
> 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
>>> 
>