ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Ozerov (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (IGNITE-2257) Incorrect deserialization of BinaryContext.
Date Thu, 24 Dec 2015 09:31:49 GMT

     [ https://issues.apache.org/jira/browse/IGNITE-2257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Vladimir Ozerov updated IGNITE-2257:
------------------------------------
    Description: 
1) Initially BinaryContext was deserialized using grid name. This is incorrect, because on
the other side grid name might be different.
{code}IgniteKernal g = IgnitionEx.gridx(gridName);{code}

2) Now it is deserialized using TLS grid name which depends on IgniteThread. This is incorrect
either because deserialization code might be invoked in user thread.
IgniteKernal g = IgnitionEx.localIgnite();

Proposed fix: set TLS BinaryContext inside GridBinaryMarshaller when deserialzation starts,
and reset it afterwards.


  was:
1) Initially BinaryContext was deserialized using grid name: 
{code}IgniteKernal g = IgnitionEx.gridx(gridName);


> Incorrect deserialization of BinaryContext.
> -------------------------------------------
>
>                 Key: IGNITE-2257
>                 URL: https://issues.apache.org/jira/browse/IGNITE-2257
>             Project: Ignite
>          Issue Type: Bug
>          Components: general, interop
>    Affects Versions: ignite-1.4
>            Reporter: Vladimir Ozerov
>            Assignee: Vladimir Ozerov
>            Priority: Blocker
>             Fix For: 1.5
>
>
> 1) Initially BinaryContext was deserialized using grid name. This is incorrect, because
on the other side grid name might be different.
> {code}IgniteKernal g = IgnitionEx.gridx(gridName);{code}
> 2) Now it is deserialized using TLS grid name which depends on IgniteThread. This is
incorrect either because deserialization code might be invoked in user thread.
> IgniteKernal g = IgnitionEx.localIgnite();
> Proposed fix: set TLS BinaryContext inside GridBinaryMarshaller when deserialzation starts,
and reset it afterwards.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message