diff options
| author | Eric Wong <normalperson@yhbt.net> | 2009-09-09 10:02:49 -0700 |
|---|---|---|
| committer | Eric Wong <normalperson@yhbt.net> | 2009-09-09 10:09:57 -0700 |
| commit | 7eb0c4e29e567f02affc202b51eb277cbae43688 (patch) | |
| tree | 59b2aa2efe2cc427cddc5638d1ccc152454419e8 /ext/clogger_ext/clogger.c | |
| parent | 72dfd64ed0ab31ced0f27e8a8a941c555221bb96 (diff) | |
| download | clogger-7eb0c4e29e567f02affc202b51eb277cbae43688.tar.gz | |
Since the wrapped Clogger object always responds to close, we cannot blindly delegate the close method to the body without ensuring it can be closed. So ensure that it can be closed before attempting to close it, all return values and errors are trapped and returned. Reported-by: Iñaki Baz Castillo
Diffstat (limited to 'ext/clogger_ext/clogger.c')
| -rw-r--r-- | ext/clogger_ext/clogger.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ext/clogger_ext/clogger.c b/ext/clogger_ext/clogger.c index 63322f9..7813c8c 100644 --- a/ext/clogger_ext/clogger.c +++ b/ext/clogger_ext/clogger.c @@ -656,7 +656,9 @@ static VALUE clogger_close(VALUE self) { struct clogger *c = clogger_get(self); - return rb_funcall(c->body, close_id, 0); + if (rb_respond_to(c->body, close_id)) + return rb_funcall(c->body, close_id, 0); + return Qnil; } /* :nodoc: */ |