X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=ext%2Fext.c;h=bf1c4f03572c08089f1df51d823fcaf4efbabedd;hb=2ab5c94a4033ecc44f0eabe58c6f792c49c9f169;hp=63096a7e744cd9cad366a946c97bf217036f630a;hpb=918d447d3b418316061a5cdeecbb4c8b64c84aa0;p=ruby-eet.git

diff --git a/ext/ext.c b/ext/ext.c
index 63096a7..bf1c4f0 100644
--- a/ext/ext.c
+++ b/ext/ext.c
@@ -1,5 +1,5 @@
 /*
- * $Id: ext.c 67 2005-06-29 15:44:09Z tilman $
+ * $Id: ext.c 71 2005-07-16 12:28:17Z tilman $
  *
  * Copyright (c) 2005 Tilman Sauerbeck (tilman at code-monkey de)
  *
@@ -47,8 +47,7 @@
 static VALUE c_close (VALUE self);
 
 static VALUE cStream, cChunk,
-             eEetError, eNameError, ePropError, eStreamError,
-             eBadElementError,
+             eEetError, eNameError, ePropError,
              sym_lossy, sym_level, sym_quality, sym_char, sym_short,
              sym_long_long, sym_double;
 static ID id_include, id_serialize, id_keys, id_pack,
@@ -453,9 +452,6 @@ stream_serialize (VALUE self)
 	for (i = 0; i < stream->len; i++) {
 		VALUE str;
 
-		if (rb_obj_is_kind_of (stream->ptr[i], cChunk) == Qfalse)
-			rb_raise (eBadElementError, "stream member is not a Chunk");
-
 		str = rb_funcall (stream->ptr[i], id_serialize, 0, NULL);
 
 		rb_ary_push (ret, str);
@@ -467,7 +463,7 @@ stream_serialize (VALUE self)
 static VALUE
 chunk_init (VALUE self, VALUE tag, VALUE data)
 {
-	unsigned long len;
+	long tag_len, data_len, tmp;
 
 	StringValue (tag);
 	StringValue (data);
@@ -478,8 +474,11 @@ chunk_init (VALUE self, VALUE tag, VALUE data)
 	/* libeet uses a signed 32bit integer to store the
 	 * chunk size, so make sure we don't overflow it
 	 */
-	len = RSTRING (tag)->len + 1 + RSTRING (data)->len;
-	if (len < 0 || len >= 2147483647L)
+	tag_len = RSTRING (tag)->len;
+	data_len = RSTRING (data)->len;
+	tmp = tag_len + 1 + data_len;
+
+	if (tmp < tag_len || tmp < data_len || tmp < 1 || tmp >= 2147483647L)
 		rb_raise (rb_eArgError, "tag or data too long");
 
 	rb_ivar_set (self, id_tag, rb_str_dup_frozen (tag));
@@ -530,8 +529,6 @@ static int
 for_each_prop (VALUE tag, VALUE arg, VALUE stream)
 {
 	VALUE value, type, tmp;
-	struct RArray *chunks;
-	long i;
 
 	if (rb_obj_is_kind_of (arg, rb_cArray) == Qfalse)
 		rb_raise (ePropError, "hash value is not an array");
@@ -543,11 +540,7 @@ for_each_prop (VALUE tag, VALUE arg, VALUE stream)
 	type = rb_ary_entry (arg, 1);
 	tmp = rb_funcall (value, id_to_eet_chunks, 2, tag, type);
 
-	Check_Type (tmp, T_ARRAY);
-	chunks = RARRAY (tmp);
-
-	for (i = 0; i < chunks->len; i++)
-		rb_ary_push (stream, chunks->ptr[i]);
+	rb_ary_concat (stream, tmp);
 
 	return ST_CONTINUE;
 }
@@ -595,6 +588,9 @@ c_to_eet (VALUE self)
 
 	args[0] = name;
 	args[1] = rb_funcall (stream, id_serialize, 0);
+
+	rb_ary_clear (stream); /* give the GC a hand... */
+
 	chunk = rb_class_new_instance (2, args, cChunk);
 
 	return rb_funcall (chunk, id_serialize, 0);
@@ -691,9 +687,6 @@ Init_eet_ext ()
 	eEetError = rb_define_class_under (m, "EetError", rb_eStandardError);
 	eNameError = rb_define_class_under (m, "NameError", eEetError);
 	ePropError = rb_define_class_under (m, "PropertyError", eEetError);
-	eStreamError = rb_define_class_under (m, "StreamError", eEetError);
-	eBadElementError = rb_define_class_under (m, "BadElementError",
-	                                          eStreamError);
 
 	id_include = rb_intern ("include?");
 	id_serialize = rb_intern ("serialize");