From f39b3207e4250cb9e3824972d25d88e9690e8070 Mon Sep 17 00:00:00 2001 From: Mihai Patrascoiu Date: Mon, 11 Jun 2018 15:22:15 +0200 Subject: [PATCH] Prevent segmentation fault when using http.gridmap --- src/XrdHttp/XrdHttpProtocol.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/XrdHttp/XrdHttpProtocol.cc b/src/XrdHttp/XrdHttpProtocol.cc index 4cfc4a62677..36a77e93cf6 100644 --- a/src/XrdHttp/XrdHttpProtocol.cc +++ b/src/XrdHttp/XrdHttpProtocol.cc @@ -355,7 +355,7 @@ int XrdHttpProtocol::GetVOMSData(XrdLink *lp) { // To set the name we pick the first CN of the certificate subject // and hope that it makes some sense, it usually does char *lnpos = strstr(SecEntity.moninfo, "/CN="); - char bufname[64], bufname2[9]; + char bufname[256], bufname2[9]; if (lnpos) { lnpos += 4; @@ -384,7 +384,7 @@ int XrdHttpProtocol::GetVOMSData(XrdLink *lp) { } if (servGMap) { - int e = servGMap->dn2user(SecEntity.moninfo, bufname, 127, 0); + int e = servGMap->dn2user(SecEntity.moninfo, bufname, sizeof(bufname), 0); if ( !e ) { TRACEI(DEBUG, " Mapping Username: " << SecEntity.moninfo << " --> " << bufname); if (SecEntity.name) free(SecEntity.name);