/*
- * $Id: ext.c 66 2005-06-29 15:42:58Z tilman $
+ * $Id: ext.c 70 2005-07-15 20:31:58Z tilman $
*
* Copyright (c) 2005 Tilman Sauerbeck (tilman at code-monkey de)
*
static VALUE
chunk_init (VALUE self, VALUE tag, VALUE data)
{
- unsigned long len;
+ long tag_len, data_len, tmp;
StringValue (tag);
StringValue (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));
struct RString *tag, *data;
tmp = rb_ivar_get (self, id_tag);
- StringValue (tmp);
tag = RSTRING (tmp);
tmp = rb_ivar_get (self, id_data);
- StringValue (tmp);
data = RSTRING (tmp);
buf_len = 9 + tag->len + data->len;
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");
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;
}
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);