You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by jk...@apache.org on 2018/03/09 23:00:03 UTC

thrift git commit: THRIFT-4024: Skip() throws TProtocolException.INVALID_DATA on unknown data types Client: js

Repository: thrift
Updated Branches:
  refs/heads/master 138da2049 -> 50862915c


THRIFT-4024: Skip() throws TProtocolException.INVALID_DATA on unknown data types
Client: js

This closes #1503


Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/50862915
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/50862915
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/50862915

Branch: refs/heads/master
Commit: 50862915ccfd5512e9982d2b75a3df99c5e7cdd3
Parents: 138da20
Author: Philip Frank <ic...@philipfrank.de>
Authored: Wed Mar 7 21:21:30 2018 +0100
Committer: James E. King III <jk...@apache.org>
Committed: Fri Mar 9 17:59:21 2018 -0500

----------------------------------------------------------------------
 lib/js/src/thrift.js | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/thrift/blob/50862915/lib/js/src/thrift.js
----------------------------------------------------------------------
diff --git a/lib/js/src/thrift.js b/lib/js/src/thrift.js
index eeae873..2b385a3 100644
--- a/lib/js/src/thrift.js
+++ b/lib/js/src/thrift.js
@@ -1438,6 +1438,9 @@ Thrift.Protocol.prototype = {
                 }
                 this.readListEnd();
                 return null;
+
+            default:
+                throw new Thrift.TProtocolException(Thrift.TProtocolExceptionType.INVALID_DATA);
         }
     }
 };