From: Daniel Kouřil <kouril@ics.muni.cz>
Date: Tue, 19 Oct 2004 14:52:35 +0000 (+0000)
Subject: Don't free any pointers returned by gridsite. Most likely this causes some memory... 
X-Git-Tag: glite-lb-server_R_0_3_2~1
X-Git-Url: http://scientific.zcu.cz/git/?a=commitdiff_plain;h=e9e081e413a0338c1b032f49e328bcf16d4add55;p=jra1mw.git

Don't free any pointers returned by gridsite. Most likely this causes some memory leaks, but obviously prevents from SIGSEGV's caused by accessing shared memory blocks. The memory (un)allocations must be reviewed later.
---

diff --git a/org.glite.lb.server/src/lb_authz.c b/org.glite.lb.server/src/lb_authz.c
index 3ade941..5641224 100644
--- a/org.glite.lb.server/src/lb_authz.c
+++ b/org.glite.lb.server/src/lb_authz.c
@@ -291,9 +291,9 @@ parse_creds(edg_wll_VomsGroups *groups, char *subject, GRSTgaclUser **gacl_user)
 
 fail:
    if (cred)
-      GRSTgaclCredFree(cred);
+      /* XXX GRSTgaclCredFree(cred); */
    if (user)
-      GRSTgaclUserFree(user);
+      /* XXX GRSTgaclUserFree(user); */
 
    return ret;
 }
@@ -365,7 +365,7 @@ delEntry(GRSTgaclAcl *acl, GRSTgaclEntry *entry)
 	    prev->next = cur->next;
 	 else
 	    acl->firstentry = cur->next;
-	 GRSTgaclEntryFree(cur);
+	 /* XXX GRSTgaclEntryFree(cur); */
 	 found = 1;
 	 break;
       }
@@ -387,7 +387,7 @@ create_cred(char *userid, int user_type, GRSTgaclCred **cred)
       if (c == NULL)
 	 return ENOMEM;
       if (!GRSTgaclCredAddValue(c, "dn", userid)) {
-	 GRSTgaclCredFree(c);
+	 /* XXX GRSTgaclCredFree(c); */
 	 return -1; /* GACL_ERR */
       }
    } else if(user_type == EDG_WLL_USER_VOMS_GROUP) {
@@ -400,7 +400,7 @@ create_cred(char *userid, int user_type, GRSTgaclCred **cred)
       *group++ = '\0';
       if (!GRSTgaclCredAddValue(c, "vo", userid) ||
 	  !GRSTgaclCredAddValue(c, "group", group)) {
-	 GRSTgaclCredFree(c);
+	 /* XXX GRSTgaclCredFree(c); */
 	 return -1; /* GACL_ERR */
       }
    } else
@@ -599,7 +599,7 @@ edg_wll_DecodeACL(char *buf, GRSTgaclAcl **result_acl)
          entry = GACLparseEntry(cur);
          if (entry == NULL)
            {
-             GRSTgaclAclFree(acl);
+             /* XXX GRSTgaclAclFree(acl); */
              xmlFreeDoc(doc);
              return EINVAL;
            }
@@ -859,7 +859,7 @@ end:
 	if (stmt) edg_wll_FreeStmt(&stmt);
 	if (acl_id) free(acl_id);
 	if (acl_str) free(acl_str);
-	if (gacl) GRSTgaclAclFree(gacl);
+	/* XXX if (gacl) GRSTgaclAclFree(gacl); */
 	if (jobstr) free(jobstr);
 
 	return edg_wll_Error(ctx, NULL, NULL);