From 7bc47c026066831295b3fdf1ed8f7f8bfafef957 Mon Sep 17 00:00:00 2001 From: akr Date: Sun, 20 Mar 2011 03:43:40 +0000 Subject: [PATCH] * ext/openssl/ossl_x509store.c: parenthesize macro arguments. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@31134 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 4 ++++ ext/openssl/ossl_x509store.c | 24 ++++++++++++------------ 2 files changed, 16 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5201df11aa..d94f0126d7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,7 @@ +Sun Mar 20 12:43:12 2011 Tanaka Akira + + * ext/openssl/ossl_x509store.c: parenthesize macro arguments. + Sun Mar 20 01:39:48 2011 Tajima Akio * hash.c (ruby_setenv): check env process block size with OS ver. diff --git a/ext/openssl/ossl_x509store.c b/ext/openssl/ossl_x509store.c index 317fc3534a..e7856485ca 100644 --- a/ext/openssl/ossl_x509store.c +++ b/ext/openssl/ossl_x509store.c @@ -11,37 +11,37 @@ #include "ossl.h" #define WrapX509Store(klass, obj, st) do { \ - if (!st) { \ + if (!(st)) { \ ossl_raise(rb_eRuntimeError, "STORE wasn't initialized!"); \ } \ - obj = Data_Wrap_Struct(klass, 0, X509_STORE_free, st); \ + (obj) = Data_Wrap_Struct((klass), 0, X509_STORE_free, (st)); \ } while (0) #define GetX509Store(obj, st) do { \ - Data_Get_Struct(obj, X509_STORE, st); \ - if (!st) { \ + Data_Get_Struct((obj), X509_STORE, (st)); \ + if (!(st)) { \ ossl_raise(rb_eRuntimeError, "STORE wasn't initialized!"); \ } \ } while (0) #define SafeGetX509Store(obj, st) do { \ - OSSL_Check_Kind(obj, cX509Store); \ - GetX509Store(obj, st); \ + OSSL_Check_Kind((obj), cX509Store); \ + GetX509Store((obj), (st)); \ } while (0) #define WrapX509StCtx(klass, obj, ctx) do { \ - if (!ctx) { \ + if (!(ctx)) { \ ossl_raise(rb_eRuntimeError, "STORE_CTX wasn't initialized!"); \ } \ - obj = Data_Wrap_Struct(klass, 0, ossl_x509stctx_free, ctx); \ + (obj) = Data_Wrap_Struct((klass), 0, ossl_x509stctx_free, (ctx)); \ } while (0) #define GetX509StCtx(obj, ctx) do { \ - Data_Get_Struct(obj, X509_STORE_CTX, ctx); \ - if (!ctx) { \ + Data_Get_Struct((obj), X509_STORE_CTX, (ctx)); \ + if (!(ctx)) { \ ossl_raise(rb_eRuntimeError, "STORE_CTX is out of scope!"); \ } \ } while (0) #define SafeGetX509StCtx(obj, storep) do { \ - OSSL_Check_Kind(obj, cX509StoreContext); \ - GetX509Store(obj, ctx); \ + OSSL_Check_Kind((obj), cX509StoreContext); \ + GetX509Store((obj), (ctx)); \ } while (0) /*