A memory leakage in mondo-2.2.9.1-1

[robert lin]

mondoarchive doesn't work on backing up Windows
A memory leakage occurrs at below line marked as ##. Just delete this could be ok.

src/common/libmondo-archive.c

:
:

default:

log_msg(2,

"feed_into_ntfsprog() called in background --- pid=%ld",
(long int) (pid));

mr_free(sz_devfile); ## delete this line

}

[Bruno Cornec]

I disagree with your analysis. Removing such a line will never allow you to solve a memory *leak* problem. Rather it will create one.

The sz_devfile var is allocated by the mr_asprintf just upper. So it needs to be freed after its usage in the parent.

So what is your problem and could you provide log files please ?

[robert lin]

please check line 18

[robert lin]

Please check the attachement. I just picked up the error part of the log.

[robert lin]

Of course, -x /dev/sda1 option was used for mondoarchive command.

[Bruno Cornec]

So your real error is *not* a memory leak, but an impossibility to open a device:

Cannot archive bigfile '/dev/sda1': not found

Now, I still need the *full* log file in order to understand what may happen. But with the above message check that you can access /dev/sda1 already.

[robert lin]

this is correct log.

[robert lin]

Please check 5023 line in mondoarchive.log.

[robert lin]

Breakpoint 1, make_slices_and_images (

biggielist_fname=0x8848c70 "/tmp/mondo.scratch.1200/mondo.scratch.2382/archives/biggielist.txt") at libmondo-archive.c:2112[[BR]]

2112 log_msg(2,
(gdb) display ntfsprog_fifo
1: ntfsprog_fifo = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
(gdb) display sz_devfile
2: sz_devfile = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
(gdb) n
2115 mr_free(sz_devfile);
2: sz_devfile = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
1: ntfsprog_fifo = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
(gdb)
2112 log_msg(2,
2: sz_devfile = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
1: ntfsprog_fifo = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
(gdb)
2115 mr_free(sz_devfile);
2: sz_devfile = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
1: ntfsprog_fifo = 0x88453d0 "/tmp/mondo.tmp.UjKSLQ/8364.441.000"
(gdb)
2124 if (IS_THIS_A_STREAMING_BACKUP(bkpinfo->backup_media_type)) {
2: sz_devfile = 0x0
1: ntfsprog_fifo = 0x88453d0 "\b\213\204\b/mondo.tmp.UjKSLQ/8364.441.000"
(gdb)
2130 res =
2: sz_devfile = 0x0
1: ntfsprog_fifo = 0x88453d0 "\b\213\204\b/mondo.tmp.UjKSLQ/8364.441.000"
(gdb)

[Bruno Cornec]

Ah, sorry, indeed there is an issue !

So sz_devfile is clearly freed too early. What about the proposed patch in rev [2578] ?

[robert lin]

Thanks so much!
It works. So, please close the issue.