[OpenAFS-devel] 1.6 "coverity" gerrit reviews

[Andrew Deason]

Hello,

There is a big stack of gerrit submissions for 1.6 right now under the
"coverity" topic. There are a lot of them, but they tend to be very
small and easy to review, so I'm asking anyone who can to just take a
few minutes and look at at least a few of them. For any change, just
take a look, check that it makes sense to you, and +1 it if it does. If
you want to help out with OpenAFS and help the 1.6.9 release move
forward, this is something you can do that doesn't take much time
(assuming that you know C).

This particular stack of changes has been getting shuffled around a
little bit to solve build errors and other things, but it's settled down
now. I believe we're trying to get this out of the way before we tackle
more substantive changes, so they shouldn't be moved around much
anymore.

Of course, review and participation for any changes are welcome; I'm
just calling this out specifically because these changes are numerous
but usually easy-to-review.

The particular stack of changes I'm talking about is every gerrit
submission between 11008 and 11065, inclusive. (Plus 11148, minus
11014.)

Or you can see most of them listed here:
<http://gerrit.openafs.org/#q,status:open+project:openafs+branch:openafs-stable-1_6_x+topic:coverity,n,z>
(that is missing http://gerrit.openafs.org/11148 because I forgot to add
it)

Or if you would like individual links to each one, here is a list, in
dependency order:

<http://gerrit.openafs.org/11008>
<http://gerrit.openafs.org/11009>
<http://gerrit.openafs.org/11010>
<http://gerrit.openafs.org/11011>
<http://gerrit.openafs.org/11012>
<http://gerrit.openafs.org/11013>
<http://gerrit.openafs.org/11015>
<http://gerrit.openafs.org/11016>
<http://gerrit.openafs.org/11017>
<http://gerrit.openafs.org/11018>
<http://gerrit.openafs.org/11019>
<http://gerrit.openafs.org/11020>
<http://gerrit.openafs.org/11021>
<http://gerrit.openafs.org/11022>
<http://gerrit.openafs.org/11023>
<http://gerrit.openafs.org/11148>
<http://gerrit.openafs.org/11024>
<http://gerrit.openafs.org/11025>
<http://gerrit.openafs.org/11026>
<http://gerrit.openafs.org/11027>
<http://gerrit.openafs.org/11028>
<http://gerrit.openafs.org/11029>
<http://gerrit.openafs.org/11030>
<http://gerrit.openafs.org/11031>
<http://gerrit.openafs.org/11032>
<http://gerrit.openafs.org/11033>
<http://gerrit.openafs.org/11034>
<http://gerrit.openafs.org/11035>
<http://gerrit.openafs.org/11036>
<http://gerrit.openafs.org/11037>
<http://gerrit.openafs.org/11038>
<http://gerrit.openafs.org/11039>
<http://gerrit.openafs.org/11040>
<http://gerrit.openafs.org/11041>
<http://gerrit.openafs.org/11042>
<http://gerrit.openafs.org/11043>
<http://gerrit.openafs.org/11044>
<http://gerrit.openafs.org/11045>
<http://gerrit.openafs.org/11046>
<http://gerrit.openafs.org/11047>
<http://gerrit.openafs.org/11048>
<http://gerrit.openafs.org/11049>
<http://gerrit.openafs.org/11050>
<http://gerrit.openafs.org/11051>
<http://gerrit.openafs.org/11052>
<http://gerrit.openafs.org/11053>
<http://gerrit.openafs.org/11054>
<http://gerrit.openafs.org/11055>
<http://gerrit.openafs.org/11056>
<http://gerrit.openafs.org/11057>
<http://gerrit.openafs.org/11058>
<http://gerrit.openafs.org/11059>
<http://gerrit.openafs.org/11060>
<http://gerrit.openafs.org/11061>
<http://gerrit.openafs.org/11062>
<http://gerrit.openafs.org/11063>
<http://gerrit.openafs.org/11064>
<http://gerrit.openafs.org/11065>

-- 
Andrew Deason
adeason@sinenomine.net