Bug #9913

Digest == throws TypeError when testing against nil

Added by Bryan Hockey 9 months ago. Updated 6 months ago.

[ruby-core:62967]
Status:Closed
Priority:Normal
Assignee:-
ruby -v:ruby 1.9.3p194 (2012-04-20 revision 35410) [x86_64-linux] Backport:2.0.0: DONE, 2.1: DONE

Description

Also present in ruby -v: jruby 1.7.10 (1.9.3p392) 2014-01-09 c4ecd6b on OpenJDK 64-Bit Server VM 1.7.0_55-b14 [linux-amd64]

To recreate:

require 'digest'
x = Digest::SHA256.new()
x == nil
TypeError: can't convert nil into String

Workaround:
w1: use #nil?
w2: reverse rhs, lhs, e.g. if nil == some_digest

Associated revisions

Revision 46368
Added by Benoit Daloze 9 months ago

digest.c: fix #== for non-string arguments

  • ext/digest/digest.c (rb_digest_instance_equal): fix #== for non-string arguments. [Bug #9913]
  • test/digest/test_digest.rb: add test for above.

Revision 46368
Added by Benoit Daloze 9 months ago

digest.c: fix #== for non-string arguments

  • ext/digest/digest.c (rb_digest_instance_equal): fix #== for non-string arguments. [Bug #9913]
  • test/digest/test_digest.rb: add test for above.

Revision 46371
Added by Nobuyoshi Nakada 9 months ago

digest.c: call to_s once

  • ext/digest/digest.c (rb_digest_instance_equal): no need to call to_s twice. [Bug #9913]

Revision 46371
Added by Nobuyoshi Nakada 9 months ago

digest.c: call to_s once

  • ext/digest/digest.c (rb_digest_instance_equal): no need to call to_s twice. [Bug #9913]

Revision 46809
Added by Tomoyuki Chikanaga 8 months ago

merge revision(s) r46368,r46371: [Backport #9913]

* ext/digest/digest.c (rb_digest_instance_equal):
  fix #== for non-string arguments.  [Bug #9913]

* test/digest/test_digest.rb: add test for above.

* ext/digest/digest.c (rb_digest_instance_equal): no need to call
  `to_s` twice.  [Bug #9913]

Revision 47333
Added by Usaku NAKAMURA 6 months ago

merge revision(s) 46368,46371: [Backport #9913]

* ext/digest/digest.c (rb_digest_instance_equal):
  fix #== for non-string arguments.  [Bug #9913]

* test/digest/test_digest.rb: add test for above.

* ext/digest/digest.c (rb_digest_instance_equal): no need to call
  `to_s` twice.  [Bug #9913]

History

#1 Updated by Benoit Daloze 9 months ago

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

Applied in changeset r46368.


digest.c: fix #== for non-string arguments

  • ext/digest/digest.c (rb_digest_instance_equal): fix #== for non-string arguments. [Bug #9913]
  • test/digest/test_digest.rb: add test for above.

#2 Updated by Tomoyuki Chikanaga 8 months ago

  • Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN to 2.0.0: REQUIRED, 2.1: REQUIRED

#3 Updated by Tomoyuki Chikanaga 8 months ago

  • Backport changed from 2.0.0: REQUIRED, 2.1: REQUIRED to 2.0.0: REQUIRED, 2.1: DONE

Backported into ruby_2_1 branch at r46809.

#4 Updated by Usaku NAKAMURA 6 months ago

  • Backport changed from 2.0.0: REQUIRED, 2.1: DONE to 2.0.0: DONE, 2.1: DONE

backported into ruby_2_0_0 at r47333.

Also available in: Atom PDF