Whamcloud - gitweb
LU-7274 gss_do_ctx_fini_rpc() passes an already freed request 74/17574/2
authorakam kumar bharathi <azurelustre@gmail.com>
Fri, 11 Dec 2015 22:27:06 +0000 (03:57 +0530)
committerOleg Drokin <oleg.drokin@intel.com>
Fri, 18 Dec 2015 05:28:19 +0000 (05:28 +0000)
commit0bcc4f6df2d5062e68e570d4daad60a39cdb5fa8
tree0dc053e17d941ac60b7b24d901289badb3153cd4
parentfdddeb25a54324cbcc2c99daf38b9f778fdbbec3
LU-7274 gss_do_ctx_fini_rpc() passes an already freed request

ptlrpc_request_free(req) calls either __ptlrpc_free_req_to_pool()
OR ptlrpc_request_cache_free() depending on request from pool /
memory.

ptlrpc_req_finished()-> __ptlrpc_req_finished()->__ptlrpc_free_req()
calls either __ptlrpc_free_req_to_pool() OR ptlrpc_request_cache_free()
too, but depending on reference and request count.

Thus its safe to removed the call to ptlrpc_request_free(req).

Change-Id: Ie2dd647af6b07d2810c458707b791b71c92bc3df
Signed-off-by: akam kumar bharathi <azurelustre@gmail.com>
Reviewed-on: http://review.whamcloud.com/17574
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ptlrpc/gss/gss_cli_upcall.c