You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Vladimir Ozerov (JIRA)" <ji...@apache.org> on 2015/12/24 10:31:49 UTC
[jira] [Updated] (IGNITE-2257) Incorrect deserialization of
BinaryContext.
[ 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)