Incidentally, stick to && and || instead of and and or. There are some
funny precedence rules.

See https://github.com/bbatsov/ruby-style-guide
On Nov 23, 2013 7:29 PM, "wvu-r7" <notifications@github.com> wrote:

> In modules/auxiliary/scanner/chargen/chargen_probe.rb:
>
> > +
> > +  def run_host(rhost)
> > +    begin
> > +      ::Timeout.timeout(to) do
> > +        connect_udp
> > +        pkt = Rex::Text.rand_text_alpha_lower(1)
> > +        req = udp_sock.write(pkt)
> > +
> > +        while (res = udp_sock.recvfrom(65535,0.1) and res[1])
> > +
> > +          if (datastore['DEBUG'])
> > +            print_status("DEBUG: #{res.to_s}")
> > +          end
> > +
> > +          res = res.to_s.strip.upcase
> > +          if (res.match(/ABCDEFGHIJKLMNOPQRSTUVWXYZ/i) or res(/0123456789/))
>
> I think you forgot a .match?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rapid7/metasploit-framework/pull/2681/files#r7877722>
> .
>