From e529cb40d072d08362f3ee68181929e671b9510e Mon Sep 17 00:00:00 2001 From: knu Date: Mon, 26 Feb 2007 06:22:08 +0000 Subject: [PATCH] * ext/thread/thread.c: Use xmalloc()/xfree() instead of malloc()/free(); pointed out by shugo in [ruby-dev:30412]. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8@11887 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ ext/thread/thread.c | 16 ++++++++-------- 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index 15fb709d13..2db57496d7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Mon Feb 26 15:18:23 2007 Akinori MUSHA + + * ext/thread/thread.c: Use xmalloc()/xfree() instead of + malloc()/free(); pointed out by shugo in [ruby-dev:30412]. + Sun Feb 25 23:02:55 2007 Akinori MUSHA * lib/test/unit/autorunner.rb (Test::Unit::AutoRunner::initialize): diff --git a/ext/thread/thread.c b/ext/thread/thread.c index c65c4bbe52..64215dca72 100644 --- a/ext/thread/thread.c +++ b/ext/thread/thread.c @@ -86,7 +86,7 @@ free_entries(Entry *first) Entry *next; while (first) { next = first->next; - free(first); + xfree(first); first = next; } } @@ -107,7 +107,7 @@ push_list(List *list, VALUE value) entry = list->entry_pool; list->entry_pool = entry->next; } else { - entry = (Entry *)malloc(sizeof(Entry)); + entry = (Entry *)xmalloc(sizeof(Entry)); } entry->value = value; @@ -325,7 +325,7 @@ free_mutex(Mutex *mutex) { assert_no_survivors(&mutex->waiting, "mutex", mutex); finalize_mutex(mutex); - free(mutex); + xfree(mutex); } static void @@ -347,7 +347,7 @@ static VALUE rb_mutex_alloc(VALUE klass) { Mutex *mutex; - mutex = (Mutex *)malloc(sizeof(Mutex)); + mutex = (Mutex *)xmalloc(sizeof(Mutex)); init_mutex(mutex); return Data_Wrap_Struct(klass, mark_mutex, free_mutex, mutex); } @@ -598,7 +598,7 @@ free_condvar(ConditionVariable *condvar) { assert_no_survivors(&condvar->waiting, "condition variable", condvar); finalize_condvar(condvar); - free(condvar); + xfree(condvar); } static void @@ -620,7 +620,7 @@ rb_condvar_alloc(VALUE klass) { ConditionVariable *condvar; - condvar = (ConditionVariable *)malloc(sizeof(ConditionVariable)); + condvar = (ConditionVariable *)xmalloc(sizeof(ConditionVariable)); init_condvar(condvar); return Data_Wrap_Struct(klass, mark_condvar, free_condvar, condvar); @@ -806,7 +806,7 @@ free_queue(Queue *queue) assert_no_survivors(&queue->space_available.waiting, "queue", queue); assert_no_survivors(&queue->value_available.waiting, "queue", queue); finalize_queue(queue); - free(queue); + xfree(queue); } static void @@ -831,7 +831,7 @@ static VALUE rb_queue_alloc(VALUE klass) { Queue *queue; - queue = (Queue *)malloc(sizeof(Queue)); + queue = (Queue *)xmalloc(sizeof(Queue)); init_queue(queue); return Data_Wrap_Struct(klass, mark_queue, free_queue, queue); }