Changeset 4620

Show
Ignore:
Timestamp:
11/11/08 10:56:47 (2 months ago)
Author:
tobias
Message:

fixing a memory leak

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/src/charon/plugins/kernel_netlink/kernel_netlink_ipsec.c

    r4618 r4620  
    17571757    int fd, family, port; 
    17581758    enumerator_t *sockets; 
     1759    bool status = TRUE; 
    17591760     
    17601761    /* we open an AF_KEY socket to autoload the af_key module. Otherwise 
    …… 
    17981799            DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s", 
    17991800                 strerror(errno)); 
    1800             return FALSE; 
     1801            status = FALSE; 
     1802            break; 
    18011803        } 
    18021804        policy.sadb_x_policy_dir = IPSEC_DIR_INBOUND; 
    …… 
    18051807            DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s",  
    18061808                 strerror(errno)); 
    1807             return FALSE; 
    1808         } 
    1809     } 
    1810     return TRUE; 
     1809            status = FALSE; 
     1810            break; 
     1811        } 
     1812    } 
     1813    sockets->destroy(sockets); 
     1814    return status; 
    18111815} 
    18121816 

  • trunk/src/charon/plugins/kernel_pfkey/kernel_pfkey_ipsec.c

    r4618 r4620  
    18801880    int fd, family, port; 
    18811881    enumerator_t *sockets; 
     1882    bool status = TRUE; 
    18821883     
    18831884    sockets = charon->socket->create_enumerator(charon->socket); 
    …… 
    19111912            DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s", 
    19121913                 strerror(errno)); 
    1913             return FALSE; 
     1914            status = FALSE; 
     1915            break; 
    19141916        } 
    19151917        policy.sadb_x_policy_dir = IPSEC_DIR_INBOUND; 
    …… 
    19181920            DBG1(DBG_KNL, "unable to set IPSEC_POLICY on socket: %s",  
    19191921                 strerror(errno)); 
    1920             return FALSE; 
    1921         } 
    1922     } 
    1923     return TRUE; 
     1922            status = FALSE; 
     1923            break; 
     1924        } 
     1925    } 
     1926    sockets->destroy(sockets); 
     1927    return status; 
    19241928} 
    19251929