Re: PR/50951 CVS commit: src/sys/fs/cd9660

[David Holland <dholland-bugs%netbsd.org@localhost>]

The following reply was made to PR kern/50951; it has been noted by GNATS.

From: David Holland <dholland-bugs%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: PR/50951 CVS commit: src/sys/fs/cd9660
Date: Sat, 12 Mar 2016 03:17:06 +0000

 On Sat, Mar 12, 2016 at 01:50:01AM +0000, David Holland wrote:
  >> -       if ((assoc = cl > 1 && *cname == ASSOCCHAR)) {
  >> +       if ((assoc = cl) > 1 && *cname == ASSOCCHAR) {
  >  
  >  
  >  ...speaking of potentially messing up, I think this one ought to be
  >  
  >          if ((assoc = (cl > 1)) && *cname == ASSOCCHAR) {
  >  
  >  based on the use of "assoc" further down in the function.
 
 or not quite.
 
 christos has it now I think.
 
 -- 
 David A. Holland
 dholland%netbsd.org@localhost