diff options
author | Sven Wegener <sven.wegener@stealer.net> | 2008-08-10 09:17:59 +0000 |
---|---|---|
committer | Sven Wegener <sven.wegener@stealer.net> | 2008-08-11 11:45:40 +0200 |
commit | 8ab19ea36c5c5340ff598e4d15fc084eb65671dc (patch) | |
tree | 031f48201a1e3a07a22a8f6734d0bb1c806b2c28 /include | |
parent | bc0fde2fad007a81ecffceb25a893a6c3f1ed767 (diff) |
ipvs: Fix possible deadlock in estimator code
There is a slight chance for a deadlock in the estimator code. We can't call
del_timer_sync() while holding our lock, as the timer might be active and
spinning for the lock on another cpu. Work around this issue by using
try_to_del_timer_sync() and releasing the lock. We could actually delete the
timer outside of our lock, as the add and kill functions are only every called
from userspace via [gs]etsockopt() and are serialized by a mutex, but better
make this explicit.
Signed-off-by: Sven Wegener <sven.wegener@stealer.net>
Cc: stable <stable@kernel.org>
Acked-by: Simon Horman <horms@verge.net.au>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions