Memory leakage

[laowei]

In init.c file, line 1577, if c->c1.tuntap is not NULL, the code doesn't free the memory. do_init_tun will alloc new memory for c->c1.tuntap.

#ifdef TARGET_ANDROID
    /* If we emulate persist-tun on android we still have to open a new tun and
     * then close the old */
    int oldtunfd = -1;
    if (c->c1.tuntap)
    {
        oldtunfd = c->c1.tuntap->fd;
    }
#endif

    /* initialize (but do not open) tun/tap object */
    do_init_tun(c);

[Gert Döring]

this is only relevant on TARGET_ANDROID, as for #ifndef TARGET_ANDROID, that whole branch is conditional on

    if (!c->c1.tuntap)
    {

... so maybe just free()ing inside the #ifdef block above? @plai?

[laowei]

Yes. It is only relevant on TARGET_ANDROID.

[plaisthos]

I am just looking again on this since I am not sure if this was fixed or not.

I am not sure what would not be freed in that case. And the current code around oldtunfd is a different but I also see no memory leak there. Keep in mind that fd is just an int and need to special free() call to release the memory.

[plaisthos]

Sorry misread where the leak is going to happen. I am preparing a patch for this.

[flichtenheld]

Fixed in commit da3f583f30a4b2be9cc5501874373fc4f627158d