[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: memory leak



Luke Howard wrote:

> >I don't know exactly the variables I should free. I think I also should
> >free "e", but when I do it and I run the application, the program is
> >aborted and this error occurs:
>
> Free result, not e, with ldap_msgfree(). This post is probably off-topic
> for openldap-devel.
>

I've checked my program and I already free result, I just forgot to put it
in the email. This are the ldap functions I use:

ld= ldap_init (ldapHost, atoi(ldapPort));
msgid = ldap_simple_bind_s (ld, radiusMgrDn, radiusMgrPw);
ldap_search_s (ld, basedn, LDAP_SCOPE_SUBTREE, filtro, NULL, 0 ,
&result);
numValues = ldap_count_entries (ld, result);
e = ldap_first_entry (ld, result);
ldapDatosRadius2 = ldap_get_values( ld, e, attrLdap_password);

And this are the parameters I free:

ldap_value_free (ldapDatosRadius);
ldap_value_free (ldapDatosRadius2);
ldap_msgfree (result);
ldap_unbind_s (ld);
/*ldap_msgfree (e);*/

so, if e musn't be freed and there is nothing left to free, then, do you
have any idea where is the memory leak comming from?

Thanks a lot

ALICIA
--------------------------------------------------------------
main ()
{
/* INICIO DE PROGRAMA */

LDAP *ld;
LDAPMessage *result, *e;
BerElement *ber;
char **ldapDatosRadius;
char **ldapDatosRadius2;
/* Cada array puede tener 5 atributos de 25 cararcteres cada uno */
char nomAtributosRadius[5][25]={"User-Name", "Password",
"SESIONES-INFOVIA"};
char nomAtributosLdap[5][25]={"uid", "userpassword", "portlimit"};
char cadena [4 * 1024];
char filtro[256];
char buffer[1024];
char attrLdap_username[CADENA_MAXIMA]={"uid"};  /* Vble donde se
almacena el nombre del atributo Ldap para el caso de User-Name */
char attrLdap_password [CADENA_MAXIMA]={"userpassword"};        /* Vble
donde se almacena el nombre del atributo atributo Ldap en el caso de
Password */
int numValues;
int numAtrib=3;                                         /* Numero de
atributos que se quieren del usuario buscado */
int k=1;
int cont, i, len, B, numIter;
int primer_atributo=1;
int msgid, rc;
struct timeval timeout;
/* Vbles que entraban por parametro en la libreria */
char loginUsuarioRadius[30]={"usuario1@ceaid2"};
char salida[4096];

/*Iniciamos bucle */
numIter = 5000;
for (B=1 ; B <= numIter ; B++)
{

/* Abrimos fichero de trazas */
logFile = fopen
("/home/redIP/LINBOX/DESARROLLO/PRUEBAS/ESQUELETO/trazas_prueba.txt",
"a");
fprintf (logFile, "\n\n\n TRAZAS INAUGURADAS (ITERACION=%d)....\n", B);

/*LDAP_INIT : CREA ESTRUCTURA LDAP */
fprintf (logFile, "******* ldap_init *******\n");
ld= ldap_init (ldapHost, atoi(ldapPort));

/*LDAP_SIMPLE_BIND: AUTENTICACION SIMPLE ASINCRONA (ldap_unbind hace
free de ld) */
fprintf (logFile, "******* ldap_simple_bind *******\n");
msgid = ldap_simple_bind_s (ld, radiusMgrDn, radiusMgrPw);

/* LDAP_SEARCH_S: REALIZA LA BUSQUEDA POR FILTRO (somos responsables de
liberar la memoria que puede reservar) */
/* Preparamos el filtro */
sprintf (filtro, "(uid=%s)", loginUsuarioRadius);
fprintf (logFile, "filtro: %s\n", filtro);

fprintf (logFile, "******* ldap_search_s *******\n");
ldap_search_s (ld, basedn, LDAP_SCOPE_SUBTREE, filtro, NULL, 0 ,
&result);

/*LDAP_COUNT_ENTRIES: NUM D ENRADAS ENCONTRADAS EN BUSQUEDA */
fprintf (logFile, "******* ldap_count_entries *******\n");
numValues = ldap_count_entries (ld, result);

/* LDAP_FIRST_ENTRY: CAPTURA DATOS DE PRIMERA ENTRADA */
fprintf (logFile, "******* ldap_first_entry *******\n");
e = ldap_first_entry (ld, result);

/*LDAP_GET_VALUES: MANEJAR VALORES DE ATRIBUTOS */
fprintf (logFile, "******* ldap_get_values *******\n");
ldapDatosRadius2 = ldap_get_values( ld, e, attrLdap_password);
if (ldapDatosRadius2 [0]!=NULL)
{
        sprintf (buffer, "Password=%s\n", ldapDatosRadius2[0]);
        strcpy (cadena, buffer);
        for (k=0; k<numAtrib; k++)
        {
                if ((strcmp(nomAtributosRadius [k], "Password") != 0) &&
(strcmp(nomAtributosRadius[k], "User-Name") != 0))
                {
                        fprintf (logFile, "******* ldap_get_value
*******\n");
                        ldapDatosRadius = ldap_get_values (ld, e,
nomAtributosLdap[k]);
                        if (ldapDatosRadius != NULL)
                        {
                                sprintf (buffer, "%s=%s",
nomAtributosRadius[k], ldapDatosRadius[0]);
                                /* Para quitarle los blancos al atributo
*/
                                cont = strlen (buffer) -1;
                                while (buffer [cont] == ' ')
                                {
                                        buffer [cont] = '\n';
                                        cont --;
                                }
                                if (primer_atributo == 1)
                                {
                                        strcat (cadena, buffer);
                                        primer_atributo = 0;
                                }
                                else
                                {
                                        strcat (cadena, ",");
                                        strcat (cadena, buffer);
                                }
                                cont = strlen (buffer);
for (i=1; ldapDatosRadius[i] != NULL; i++)
                                {
                                        fprintf (logFile, "Es un
atributo multi-value, ");
                                        sprintf (buffer, "%s=%s, ",
nomAtributosRadius[k], ldapDatosRadius[i]);
                                        strcat (cadena, buffer );
                                        fprintf (logFile, "NEW: Por
ahora la cadena es . %s", cadena);
                                }
                        }
                }
        }
        len = strlen(cadena);
        strcpy (salida, cadena);
        fprintf (logFile, "\nEl vector de respuesta se ha rellenado:
%s", salida);
        }

fprintf (logFile, "******* ldap_value_free *******\n");
ldap_value_free (ldapDatosRadius);
fprintf (logFile, "******* ldap_values_free *******\n");
ldap_value_free (ldapDatosRadius2);
fprintf (logFile, "******* ldap_msgfree *******\n");
ldap_msgfree (result);
fprintf (logFile, "******* ldap_msgfree *******\n");
/*ldap_msgfree (e);*/
fprintf (logFile, "Se ha cerrado la conexion\n");
ldap_unbind_s (ld);
fprintf (logFile, "Ahora devolveriamos al radius : %s\n", salida);

if (B==numIter)
        fprintf (logFile, "---->>  SALGO DEL PROGRAMA POR MI PROPIO
PIE!!\n");
/* Cerramos el fichero de trazas */
fclose (logFile);

}
/* fin del bucle*/


/*FIN DEL PROGRAMA */
}
----------------------------------------------------------------------

>
> -- Luke
>
> --
> Luke Howard | lukehoward.com
> PADL Software | www.padl.com