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(+)
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
Show replies by date
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(a)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(a)redhat.com>
Reviewed-by: Simo Sorce <simo(a)redhat.com>
Reviewed-by: Robbie Harwood <rharwood(a)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;
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
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(a)redhat.com>
Date: Tue Aug 8 14:12:40 2017 -0400
Add call to verto_cleanup()
Signed-off-by: Alexander Scheel <ascheel(a)redhat.com>
Reviewed-by: Simo Sorce <simo(a)redhat.com>
Reviewed-by: Robbie Harwood <rharwood(a)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;
}
--
To stop receiving notification emails like this one, please contact
the administrator of this repository.