]> git.michaelhowe.org Git - packages/o/openafs.git/commit
rx: Assert call error for RXS_PreparePacket error
authorAndrew Deason <adeason@sinenomine.net>
Mon, 18 Feb 2013 01:34:06 +0000 (19:34 -0600)
committerStephan Wiesand <stephan.wiesand@desy.de>
Sat, 30 Mar 2013 15:11:34 +0000 (08:11 -0700)
commitd16fac9dfb55cc70bc9f868ee58db9ba143108e9
tree2835a314462bcf8aff6af1e284f4b5c815171cec
parent997a9980e33ae92abdaa81d07d0857d41dd6e9ae
rx: Assert call error for RXS_PreparePacket error

If we've received an error from the underlying security class, we must
not try to send the given packet, or we risk security issues. We
currently achieve this by setting an error on the connection. It is
slightly indirect in how this yields an error on this specific call,
and so it may not be immediately clear, but doing so is critical. If
somehow the call does not have an error by the end of this, we cannot
proceed as this is an error condition we do not handle. So, assert.

Reviewed-on: http://gerrit.openafs.org/9122
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
(cherry picked from commit 85f2a66ceafd7a13f51d0352c62b5d69f4620edb)

Change-Id: Iae523e6f18dd73749a6be5c3d10e132e5c14a70c
Reviewed-on: http://gerrit.openafs.org/9280
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/rx/rx_packet.c