Re: svn commit: r253274 - head/sys/cam/scsi

2013-07-15 Thread Ulrich Spörlein
2013/7/15 Kenneth D. Merry : > Oops, you're right! Thanks for pointing it out! I just committed a fix. > > How does Coverity detect something like that? Using the comment, or the > variable name? I would guess the variable name, maybe with a fuzzy match. At least that's how I would do it, I don

Re: svn commit: r253274 - head/sys/cam/scsi

2013-07-15 Thread Kenneth D. Merry
On Mon, Jul 15, 2013 at 17:14:00 +0200, Ulrich Sp??rlein wrote: > On Fri, 2013-07-12 at 17:09:50 +, Kenneth D. Merry wrote: > > Author: ken > > Date: Fri Jul 12 17:09:50 2013 > > New Revision: 253274 > > URL: http://svnweb.freebsd.org/changeset/base/253274 > > > > Log: > > Fix a problem with

Re: svn commit: r253274 - head/sys/cam/scsi

2013-07-15 Thread Ulrich Spörlein
On Fri, 2013-07-12 at 17:09:50 +, Kenneth D. Merry wrote: > Author: ken > Date: Fri Jul 12 17:09:50 2013 > New Revision: 253274 > URL: http://svnweb.freebsd.org/changeset/base/253274 > > Log: > Fix a problem with READ ELEMENT STATUS that occurs on some > changers that don't support the DVC