This is an automated email from the git hooks/post-receive script.
rharwood pushed a change to branch master in repository gssproxy.
from 8ba0f42 Prevent uninitialized read in error path of XDR contexts new e8ad339 Don't leak mech_type when CONTINUE_NEEDED from init_sec_context new 03788ef Add call to verto_cleanup()
The 2 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "adds" were already present in the repository and have only been added to this reference.
Summary of changes: proxy/configure.ac | 6 ++++++ proxy/src/gp_export.c | 1 + proxy/src/gssproxy.c | 4 ++++ 3 files changed, 11 insertions(+)
This is an automated email from the git hooks/post-receive script.
rharwood pushed a commit to branch master in repository gssproxy.
commit e8ad339fda006f9317b456abc5de0de2080e495b Author: Alexander Scheel ascheel@redhat.com Date: Tue Aug 8 11:36:56 2017 -0400
Don't leak mech_type when CONTINUE_NEEDED from init_sec_context
Signed-off-by: Alexander Scheel ascheel@redhat.com Reviewed-by: Simo Sorce simo@redhat.com Reviewed-by: Robbie Harwood rharwood@redhat.com Merges: #207 --- proxy/src/gp_export.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/proxy/src/gp_export.c b/proxy/src/gp_export.c index ab08bb7..2e3dd07 100644 --- a/proxy/src/gp_export.c +++ b/proxy/src/gp_export.c @@ -686,6 +686,7 @@ export: case EXP_CTX_PARTIAL: /* this happens only when a init_sec_context call returns a partially * initialized context so we return only what we have, not much */ + xdr_free((xdrproc_t)xdr_gssx_OID, (char *)&out->mech); ret = gp_conv_oid_to_gssx(mech, &out->mech); if (ret) { ret_maj = GSS_S_FAILURE;
This is an automated email from the git hooks/post-receive script.
rharwood pushed a commit to branch master in repository gssproxy.
commit 03788efcb26660ba6f54182ec9e7d03fbcbd12e3 Author: Alexander Scheel ascheel@redhat.com Date: Tue Aug 8 14:12:40 2017 -0400
Add call to verto_cleanup()
Signed-off-by: Alexander Scheel ascheel@redhat.com Reviewed-by: Simo Sorce simo@redhat.com Reviewed-by: Robbie Harwood rharwood@redhat.com Merges: #207 --- proxy/configure.ac | 6 ++++++ proxy/src/gssproxy.c | 4 ++++ 2 files changed, 10 insertions(+)
diff --git a/proxy/configure.ac b/proxy/configure.ac index bc73dd9..81323ee 100644 --- a/proxy/configure.ac +++ b/proxy/configure.ac @@ -82,6 +82,12 @@ if test x$have_libverto = x; then AC_MSG_ERROR([Could not find VERTO headers]) fi
+AC_CHECK_LIB(verto, verto_free,, + AC_MSG_ERROR(["Error: libverto is not found or lacks verto_free"])) + +AC_CHECK_FUNCS(verto_cleanup,, + AC_MSG_WARN(["Warning: libverto lacks verto_cleanup function"])) + #Check for libini_config PKG_CHECK_MODULES([LIBINI_CONFIG], [ini_config >= 1.2.0], [have_libini_config=1], [have_libini_config=]) if test x$have_libini_config = x; then diff --git a/proxy/src/gssproxy.c b/proxy/src/gssproxy.c index 2461f41..6d36a5d 100644 --- a/proxy/src/gssproxy.c +++ b/proxy/src/gssproxy.c @@ -299,5 +299,9 @@ cleanup: free(opt_config_dir); free(opt_config_socket);
+#ifdef HAVE_VERTO_CLEANUP + verto_cleanup(); +#endif + return ret; }
gss-proxy@lists.fedorahosted.org
-
git repository hosting