Project

General

Profile

Backport #484

OpenSSL::Config does not freeze correctly

Added by premchai21 (Drake Wilson) almost 11 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Normal
[ruby-core:18377]

Description

=begin

#!/usr/bin/ruby
require 'openssl'
c = OpenSSL::Config.new
c['foo'] = [['key', 'value']]
c.freeze
c['foo'] = [['key', 'wrong']]
p c['foo']['key']
# Prints "wrong"

=end

Associated revisions

Revision 91b056e1
Added by wyhaines (Kirk Haines) about 9 years ago

Backport #484 [ruby-core:18377]; Make OpenSSL::Config freeze correctly, preventing further config changes.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8_6@28002 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

History

#1

Updated by premchai21 (Drake Wilson) almost 11 years ago

=begin
Amazing. The preview showed me a completely wrong rendition. Let's try that again.

#!/usr/bin/ruby
require 'openssl'
c = OpenSSL::Config.new
c['foo'] = [ ['key', 'value'] ]
c.freeze
c['foo'] = [ ['key', 'wrong'] ]
p c['foo']['key'] # Prints "wrong"

=end

#2

Updated by shyouhei (Shyouhei Urabe) over 10 years ago

  • Assignee set to gotoyuzo (GOTOU Yuuzou)

=begin

=end

#3

Updated by marcandre (Marc-Andre Lafortune) almost 10 years ago

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

=begin
Applied in changeset r25069.
=end

#4

Updated by marcandre (Marc-Andre Lafortune) almost 10 years ago

  • Category set to lib
  • Target version set to Ruby 1.8.8
  • ruby -v set to -

=begin
Fixed also for Ruby 1.9 in r25017
=end

#5

Updated by shyouhei (Shyouhei Urabe) over 9 years ago

  • Status changed from Closed to Assigned
  • Assignee changed from gotoyuzo (GOTOU Yuuzou) to wyhaines (Kirk Haines)

=begin
1.8.6 has this issue, meseems.
=end

#6

Updated by wyhaines (Kirk Haines) about 9 years ago

  • Status changed from Assigned to Closed

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

=end

Also available in: Atom PDF