Mailing-List: contact dev-help@thrift.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@thrift.apache.org
Date: Tue, 10 Jan 2017 16:22:58 +0000 (UTC)
From: "Michael Antipin (JIRA)" <jira@apache.org>
To: dev@thrift.apache.org
Message-ID: <JIRA.13033349.1484048135000.698204.1484065378358@Atlassian.JIRA>
In-Reply-To: <JIRA.13033349.1484048135000@Atlassian.JIRA>
References: <JIRA.13033349.1484048135000@Atlassian.JIRA> <JIRA.13033349.1484048135178@arcas>
Subject: [jira] [Commented] (THRIFT-4024) C# deserialization takes
 unnecessary time on list with unknown type of elements
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Tue, 10 Jan 2017 16:23:00 -0000


    [ https://issues.apache.org/jira/browse/THRIFT-4024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15815406#comment-15815406 ] 

Michael Antipin commented on THRIFT-4024:
-----------------------------------------

For the time being, I use a pretty lame quick fix in TBinaryProtocol:

{code}
		public override TList ReadListBegin()
		{
			TList list = new TList();
			list.ElementType = ReadListElementType();
			list.Count = ReadI32();
			return list;
		}

		TType ReadListElementType()
		{
			var type = ReadByte();
			if (type < 0 || type > 15 || type == 5) {
				throw(new InvalidDataException("Invalid list element type: "+type));
			}
			return (TType)type;
		}
{code}

> C# deserialization takes unnecessary time on list with unknown type of elements
> -------------------------------------------------------------------------------
>
>                 Key: THRIFT-4024
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4024
>             Project: Thrift
>          Issue Type: Bug
>          Components: C# - Library
>    Affects Versions: 0.10.0
>            Reporter: Michael Antipin
>            Assignee: Jens Geyer
>
> I'm using TBinaryProtocol and a simple transport that reads from a given byte array.
> C# library contains the following code in TProtocolUtil.Skip(TProtocol prot, TType type):
> {code}
> case TType.List:
> 	TList list = prot.ReadListBegin();
> 	for (int i = 0; i < list.Count; i++) {
> 		Skip(prot, list.ElementType);
> 	}
> 	prot.ReadListEnd();
> 	break;
> {code}
> The type of elements is detected in ReadListBegin(), and, as Skip() does nothing for unknown types, the position in the binary remains the same until the for loop completes. 
> So, when you try to deserialize invalid data, and a field type happens to be detected as TType.List, you may end up waiting for a random period of time until deserialization is completed (734707176 iterations of skipping in my case).
> I suggest throwing an exception immediately when list elements type is unknown. May be, it would be good to have a setting like *FailOnUnknownType*, so that Skip() will throw instead of ignoring.


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