Project

General

Profile

Actions

Bug #5468

closed

IPSocket#valid_v6? in ipaddr.rb contains incorrect regexps

Added by jamesotron (James Harton) over 12 years ago. Updated over 6 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.1.0]
[ruby-core:40254]

Description

Code to reproduce:

require 'ipaddr'
IPAddr.new('FF32:00FF:A12:34FF:FE56:7890::/96', Socket::AF_INET) # See RFC4489

Fails with:

ArgumentError: invalid address
	from /Users/jnh/.rvm/rubies/ruby-1.9.2-p290/lib/ruby/1.9.1/ipaddr.rb:477:in `rescue in initialize'
	from /Users/jnh/.rvm/rubies/ruby-1.9.2-p290/lib/ruby/1.9.1/ipaddr.rb:474:in `initialize'
	from (irb):2:in `new'
	from (irb):2
	from /Users/jnh/.rvm/rubies/ruby-1.9.2-p290/bin/irb:16:in `<main>'

I have tested on both 1.9.2 and 1.8.7:

ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.1.0]
ruby 1.8.7 (2011-06-30 patchlevel 352) [i686-darwin11.2.0]

Looking over #valid_v6 it appears that it contains a number of invalid regular expressions (IPv4 compatible IPv6 addressing was deprecated by RFC4291 in 2006 for example). I would suggest that all the regular expressions are removed and the operating systems inet_pton() call is used to validate it instead.

Updated by kernigh (George Koehler) over 12 years ago

=begin
'FF32:00FF:A12:34FF:FE56:7890::/96' is an IPv6 prefix, but Socket::AF_INET means IPv4 address. It works if I say Socket::AF_INET6.

$ ruby -v
ruby 1.9.4dev (2011-10-18 trunk 33473) [x86_64-openbsd4.9]
$ irb
irb(main):001:0> require 'ipaddr'
=> true
irb(main):002:0> IPAddr.new('FF32:00FF:A12:34FF:FE56:7890::/96', Socket::AF_INET)
ArgumentError: address family mismatch
from /home/kernigh/prefix/lib/ruby/1.9.1/ipaddr.rb:510:in initialize' from (irb):2:in new'
from (irb):2
from /home/kernigh/prefix/bin/irb:12:in `'
irb(main):003:0> IPAddr.new('FF32:00FF:A12:34FF:FE56:7890::/96', Socket::AF_INET6)
=> #<IPAddr: IPv6:ff32:00ff:0a12:34ff:fe56:7890:0000:0000/ffff:ffff:ffff:ffff:ffff:ffff:0000:0000>
=end

Updated by jamesotron (James Harton) over 12 years ago

Apologies, you're right. That was a stupid mistake to make :)
However, I'm still getting an invalid address:

ruby-1.9.3-p0 :004 > IPAddr.new('FF32:00FF:A12:34FF:FE56:7890::/96', Socket::AF_INET6)
ArgumentError: invalid address
	from /Users/jnh/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/1.9.1/ipaddr.rb:496:in `rescue in initialize'
	from /Users/jnh/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/1.9.1/ipaddr.rb:493:in `initialize'
	from (irb):4:in `new'
	from (irb):4
	from /Users/jnh/.rvm/rubies/ruby-1.9.3-p0/bin/irb:16:in `<main>'

ruby 1.9.3p0 (2011-10-30 revision 33570) [x86_64-darwin11.2.0]

I'm presuming this is fixed in 1.9.4 then?

Updated by ko1 (Koichi Sasada) about 12 years ago

  • Status changed from Open to Assigned
  • Assignee set to knu (Akinori MUSHA)
Actions #4

Updated by naruse (Yui NARUSE) about 11 years ago

  • Target version changed from 1.9.2 to 2.6

Updated by knu (Akinori MUSHA) over 6 years ago

  • Status changed from Assigned to Closed

The parser has had updates and the given example IPv6 address in the above comment currently passes.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0