Bug #3384: ext/openssl: avoid to use cast

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: ruby 1.9.2dev (2010-05-31 revision 28117) [i486-netbsdelf]

[taca (Takahiro Kambe)]

=begin
ext/opensslで、この部分だけ旧来のcastを使用しているのは何か理由があるのでしょうか?

--- ext/openssl/ossl_ssl.c.orig 2010-05-23 12:18:51.000000000 +0000
+++ ext/openssl/ossl_ssl.c
@@ -700,10 +700,10 @@ ossl_sslctx_get_ciphers(VALUE self)
if (!ciphers)
return rb_ary_new();

• num = sk_num((STACK*)ciphers);

• num = sk_SSL_CIPHER_num(ciphers);
  ary = rb_ary_new2(num);
  for(i = 0; i < num; i++){

•    cipher = (SSL_CIPHER*)sk_value((STACK*)ciphers, i);
  

•    cipher = sk_SSL_CIPHER_value(ciphers, i);
     rb_ary_push(ary, ossl_ssl_cipher_to_ary(cipher));
  

  }
  return ary;
  =end

[mame (Yusuke Endoh)]

=begin
遠藤です。

ext/opensslで、この部分だけ旧来のcastを使用しているのは何か理由があるのでしょうか?

cast を使用していると問題があるでしょうか。
コンパイラが警告を出す程度の軽微な問題だとしたら、今は下手にいじりたく
ないなと思います。

--
Yusuke Endoh [e-mail]
=end

[naruse (Yui NARUSE)]

=begin
他の部分は変えないとコンパイルが通らなかったのに対し、ここは変えなくても通ったからじゃないかなぁと思います。
=end

[taca (Takahiro Kambe)]

=begin

cast を使用していると問題があるでしょうか。
そう言われてしまうと身も蓋もありません。本来のOpenSSLが提供しているAPIを避けて、キャストを使い続けるのはどうなんだろうと思った次第です。将来のOpenSSLのリリースに向けた移植性という面では確実に不利ではないでしょうか?

今は下手にいじりたくないなと思います。
1.9.2のリリースまでに修正して欲しいとも思ってはいません。

=end

[naruse (Yui NARUSE)]

=begin
http://cvs.openssl.org/fileview?f=openssl/crypto/stack/safestack.h&v=1.14
で入ったAPIのようなので、単に古いまま残っているだけのようですね。
なのでtrunkには取込みます。
=end

[naruse (Yui NARUSE)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

=begin
This issue was solved with changeset r28282.
Takahiro, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

=end