commit 32776c0f46f5fd79b296391d66236c23b20af072
author Dmytro Shteflyuk <kpumuk@kpumuk.info> Tue Feb 10 12:25:07 2026 -0500
committer Jens Geyer <Jens-G@users.noreply.github.com> Sat Feb 21 10:15:18 2026 +0100
tree 07867002f4461ef152060deb8a4e07dd2134102f
parent ca0740a0b9cd0a0ddeecdcdaf655f8b336c5ca75

Make sure we always throw ProtocolException(INVALID_DATA) in Ruby

lib/rb/ext/struct.c

diff --git a/lib/rb/ext/struct.c b/lib/rb/ext/struct.c
index c36c161..b61b995 100644
--- a/lib/rb/ext/struct.c
+++ b/lib/rb/ext/struct.c
@@ -20,6 +20,7 @@
 #include "struct.h"
 #include "constants.h"
 #include "macros.h"
+#include "protocol.h"
 #include "strlcpy.h"
 
 VALUE thrift_union_class;
@@ -656,7 +657,7 @@
   field_type = FIX2INT(field_type_value);
 
   if (field_type != TTYPE_STOP) {
-    rb_raise(rb_eRuntimeError, "too many fields in union!");
+    rb_exc_raise(get_protocol_exception(INT2FIX(PROTOERR_INVALID_DATA), rb_str_new2("too many fields in union!")));
   }
 
   // read struct end
@@ -684,7 +685,7 @@
   VALUE field_info = rb_hash_aref(struct_fields, field_id);
 
   if(NIL_P(field_info)) {
-    rb_raise(rb_eRuntimeError, "set_field is not valid for this union!");
+    rb_exc_raise(get_protocol_exception(INT2FIX(PROTOERR_INVALID_DATA), rb_str_new2("set_field is not valid for this union!")));
   }
 
   VALUE ttype_value = rb_hash_aref(field_info, type_sym);