Re: [jamboost] memory leakage?

From: Vladimir Prus (ghost_at_[hidden])
Date: 2002-07-03 08:43:03

David Abrahams wrote:
> Hi Vladimir,
>
> Did you ever pursue that memory issue any further? I have a feeling there
> are some massive performance gains to be had somewhere.

I've just fixed a memory leak introduced by somebody, appearently some ghost 
;-), in expand.c. Will commit shortly. But...

1. We still have memory leaks.... see the attached file. The top memory leak 
is that all strings are allocated using newstr but never freed.

2. It would be possible to free all the memory in the end, but I doubt that 
peak memory consumption can be much reduced -- many things are just allocated 
and not reference counted in any way.

- Volodya --------------Boundary-00=_R3EOX6YH2S23RKK903L9 Content-Type: text/plain;
charset="iso-8859-1";
name="leaks"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="leaks"

PT0zMDcyMT09IHZhbGdyaW5kLTEuMHByZTIsIGEgbWVtb3J5IGVycm9yIGRldGVjdG9yIGZv
ciB4ODYgR05VL0xpbnV4Lgo9PTMwNzIxPT0gQ29weXJpZ2h0IChDKSAyMDAwLTIwMDIsIGFu
ZCBHTlUgR1BMJ2QsIGJ5IEp1bGlhbiBTZXdhcmQuCj09MzA3MjE9PSBFc3RpbWF0ZWQgQ1BV
IGNsb2NrIHJhdGUgaXMgMTQ2MSBNSHoKPT0zMDcyMT09IEZvciBtb3JlIGRldGFpbHMsIHJl
cnVuIHdpdGg6IC12Cj09MzA3MjE9PSAKPT0zMDcyMT09IAo9PTMwNzIxPT0gRVJST1IgU1VN
TUFSWTogMCBlcnJvcnMgZnJvbSAwIGNvbnRleHRzIChzdXBwcmVzc2VkOiA2NzIgZnJvbSAy
KQo9PTMwNzIxPT0gbWFsbG9jL2ZyZWU6IGluIHVzZSBhdCBleGl0OiAyNjcyNDAyIGJ5dGVz
IGluIDEzMzAwNiBibG9ja3MuCj09MzA3MjE9PSBtYWxsb2MvZnJlZTogMzI0NzEyIGFsbG9j
cywgMTkxNzA2IGZyZWVzLCAyODM4Mzc0NiBieXRlcyBhbGxvY2F0ZWQuCj09MzA3MjE9PSBG
b3IgY291bnRzIG9mIGRldGVjdGVkIGVycm9ycywgcmVydW4gd2l0aDogLXYKPT0zMDcyMT09
IHNlYXJjaGluZyBmb3IgcG9pbnRlcnMgdG8gMTMzMDA2IG5vdC1mcmVlZCBibG9ja3MuCj09
MzA3MjE9PSBjaGVja2VkIDYyMDIwMjAgYnl0ZXMuCj09MzA3MjE9PSAKPT0zMDcyMT09IGRl
ZmluaXRlbHkgbG9zdDogNzE4MjcwIGJ5dGVzIGluIDIzMjMwIGJsb2Nrcy4KPT0zMDcyMT09
IHBvc3NpYmx5IGxvc3Q6ICAgMTE5MyBieXRlcyBpbiAxNiBibG9ja3MuCj09MzA3MjE9PSBz
dGlsbCByZWFjaGFibGU6IDE5NTI5MzkgYnl0ZXMgaW4gMTA5NzYwIGJsb2Nrcy4KPT0zMDcy
MT09IAo9PTMwNzIxPT0gNjAgYnl0ZXMgaW4gNSBibG9ja3MgYXJlIHBvc3NpYmx5IGxvc3Qg
aW4gbG9zcyByZWNvcmQgMiBvZiAyNQo9PTMwNzIxPT0gICAgYXQgMHg0MDA0M0JEQjogbWFs
bG9jICh2Z19jbGllbnRmdW5jcy5jOjEwMCkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0RTAyQTog
YWRkc2V0dGluZ3MgKHJ1bGVzLmM6MjM1KQo9PTMwNzIxPT0gICAgYnkgMHg4MDQ5RURBOiBj
b21waWxlX2xvY2FsIChjb21waWxlLmM6NDYxKQo9PTMwNzIxPT0gICAgYnkgMHg4MDREREJD
OiBwYXJzZV9ldmFsdWF0ZSAocGFyc2UuYzoxMzgpCj09MzA3MjE9PSAgICBieSAweDgwNDlG
MTk6IGNvbXBpbGVfbG9jYWwgKGNvbXBpbGUuYzo0NzApCj09MzA3MjE9PSAgICBieSAweDgw
NEREQkM6IHBhcnNlX2V2YWx1YXRlIChwYXJzZS5jOjEzOCkKPT0zMDcyMT09ICAgIGJ5IDB4
ODA0OUYxOTogY29tcGlsZV9sb2NhbCAoY29tcGlsZS5jOjQ3MCkKPT0zMDcyMT09IAo9PTMw
NzIxPT0gMjcyIGJ5dGVzIGluIDE3IGJsb2NrcyBhcmUgZGVmaW5pdGVseSBsb3N0IGluIGxv
c3MgcmVjb3JkIDQgb2YgMjUKPT0zMDcyMT09ICAgIGF0IDB4NDAwNDNCREI6IG1hbGxvYyAo
dmdfY2xpZW50ZnVuY3MuYzoxMDApCj09MzA3MjE9PSAgICBieSAweDgwNEE2QzU6IGV2YWx1
YXRlX3J1bGUgKGNvbXBpbGUuYzo4NDYpCj09MzA3MjE9PSAgICBieSAweDgwNEEwNzI6IGNv
bXBpbGVfcnVsZSAoY29tcGlsZS5jOjU2MykKPT0zMDcyMT09ICAgIGJ5IDB4ODA0RERCQzog
cGFyc2VfZXZhbHVhdGUgKHBhcnNlLmM6MTM4KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBODRG
OiBjb21waWxlX3J1bGVzIChjb21waWxlLmM6OTA4KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRE
REJDOiBwYXJzZV9ldmFsdWF0ZSAocGFyc2UuYzoxMzgpCj09MzA3MjE9PSAgICBieSAweDgw
NDlGMTk6IGNvbXBpbGVfbG9jYWwgKGNvbXBpbGUuYzo0NzApCj09MzA3MjE9PSAKPT0zMDcy
MT09IDExMzMgYnl0ZXMgaW4gMTEgYmxvY2tzIGFyZSBwb3NzaWJseSBsb3N0IGluIGxvc3Mg
cmVjb3JkIDcgb2YgMjUKPT0zMDcyMT09ICAgIGF0IDB4NDAwNDNCREI6IG1hbGxvYyAodmdf
Y2xpZW50ZnVuY3MuYzoxMDApCj09MzA3MjE9PSAgICBieSAweDgwNERBMkI6IG5ld3N0ciAo
bmV3c3RyLmM6NTMpCj09MzA3MjE9PSAgICBieSAweDgwNEI0MzA6IHZhcl9leHBhbmQgKGV4
cGFuZC5jOjQyNikKPT0zMDcyMT09ICAgIGJ5IDB4ODA0OUUyNjogY29tcGlsZV9saXN0IChj
b21waWxlLmM6NDI3KQo9PTMwNzIxPT0gICAgYnkgMHg4MDREREJDOiBwYXJzZV9ldmFsdWF0
ZSAocGFyc2UuYzoxMzgpCj09MzA3MjE9PSAgICBieSAweDgwNEE4NzM6IGNvbXBpbGVfc2V0
IChjb21waWxlLmM6OTI0KQo9PTMwNzIxPT0gICAgYnkgMHg4MDREREJDOiBwYXJzZV9ldmFs
dWF0ZSAocGFyc2UuYzoxMzgpCj09MzA3MjE9PSAKPT0zMDcyMT09IDE0ODggYnl0ZXMgaW4g
MTI0IGJsb2NrcyBhcmUgZGVmaW5pdGVseSBsb3N0IGluIGxvc3MgcmVjb3JkIDkgb2YgMjUK
PT0zMDcyMT09ICAgIGF0IDB4NDAwNDNCREI6IG1hbGxvYyAodmdfY2xpZW50ZnVuY3MuYzox
MDApCj09MzA3MjE9PSAgICBieSAweDgwNERGQjA6IGFjdGlvbmxpc3QgKHJ1bGVzLmM6MTg2
KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBNzM1OiBldmFsdWF0ZV9ydWxlIChjb21waWxlLmM6
ODU2KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBMDcyOiBjb21waWxlX3J1bGUgKGNvbXBpbGUu
Yzo1NjMpCj09MzA3MjE9PSAgICBieSAweDgwNEREQkM6IHBhcnNlX2V2YWx1YXRlIChwYXJz
ZS5jOjEzOCkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0OUMyQjogY29tcGlsZV9pZiAoY29tcGls
ZS5jOjMxMCkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0RERCQzogcGFyc2VfZXZhbHVhdGUgKHBh
cnNlLmM6MTM4KQo9PTMwNzIxPT0gCj09MzA3MjE9PSAxODQ4IGJ5dGVzIGluIDE1NCBibG9j
a3MgYXJlIGRlZmluaXRlbHkgbG9zdCBpbiBsb3NzIHJlY29yZCAxMCBvZiAyNQo9PTMwNzIx
PT0gICAgYXQgMHg0MDA0M0JEQjogbWFsbG9jICh2Z19jbGllbnRmdW5jcy5jOjEwMCkKPT0z
MDcyMT09ICAgIGJ5IDB4ODA0QzAyRjogbGlzdF9uZXcgKGxpc3RzLmM6ODMpCj09MzA3MjE9
PSAgICBieSAweDgwNEI3Rjk6IHZhcl9leHBhbmRfdW5pdF90ZXN0IChleHBhbmQuYzo2NTYp
Cj09MzA3MjE9PSAgICBieSAweDgwNDkwRTA6IHJ1bl91bml0X3Rlc3RzIChqYW0uYzoxOTIp
Cj09MzA3MjE9PSAgICBieSAweDgwNDkzREM6IG1haW4gKGphbS5jOjMwNikKPT0zMDcyMT09
ICAgIGJ5IDB4NDAyNjAxNEY6ICh3aXRoaW4gL2xpYi9saWJjLTIuMi41LnNvKQo9PTMwNzIx
PT0gICAgYnkgMHg4MDQ5MDExOiAod2l0aGluIC90bXAvYm9vc3QvdG9vbHMvYnVpbGQvamFt
X3NyYy9iaW4ubGludXh4ODYvYmphbSkKPT0zMDcyMT09IAo9PTMwNzIxPT0gODkwNCBieXRl
cyBpbiA3NDIgYmxvY2tzIGFyZSBkZWZpbml0ZWx5IGxvc3QgaW4gbG9zcyByZWNvcmQgMTUg
b2YgMjUKPT0zMDcyMT09ICAgIGF0IDB4NDAwNDNCREI6IG1hbGxvYyAodmdfY2xpZW50ZnVu
Y3MuYzoxMDApCj09MzA3MjE9PSAgICBieSAweDgwNERGNzQ6IHRhcmdldGVudHJ5IChydWxl
cy5jOjE2NikKPT0zMDcyMT09ICAgIGJ5IDB4ODA0REY0NzogdGFyZ2V0bGlzdCAocnVsZXMu
YzoxNDYpCj09MzA3MjE9PSAgICBieSAweDgwNTAyMjE6IGJ1aWx0aW5fZGVwZW5kcyAoYnVp
bHRpbnMuYzoyMDkpCj09MzA3MjE9PSAgICBieSAweDgwNEREQkM6IHBhcnNlX2V2YWx1YXRl
IChwYXJzZS5jOjEzOCkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0QTc4MDogZXZhbHVhdGVfcnVs
ZSAoY29tcGlsZS5jOjg2OSkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0QTA3MjogY29tcGlsZV9y
dWxlIChjb21waWxlLmM6NTYzKQo9PTMwNzIxPT0gCj09MzA3MjE9PSAxNDkxNiBieXRlcyBp
biAzMzkgYmxvY2tzIGFyZSBkZWZpbml0ZWx5IGxvc3QgaW4gbG9zcyByZWNvcmQgMTcgb2Yg
MjUKPT0zMDcyMT09ICAgIGF0IDB4NDAwNDNCREI6IG1hbGxvYyAodmdfY2xpZW50ZnVuY3Mu
YzoxMDApCj09MzA3MjE9PSAgICBieSAweDgwNERDOTU6IHBhcnNlX21ha2UgKHBhcnNlLmM6
ODIpCj09MzA3MjE9PSAgICBieSAweDgwNTFGRjk6IHl5cGFyc2UgKGphbWdyYW0ueTozNjYp
Cj09MzA3MjE9PSAgICBieSAweDgwNERDM0Y6IHBhcnNlX2ZpbGUgKHBhcnNlLmM6NTYpCj09
MzA3MjE9PSAgICBieSAweDgwNDk2Njc6IG1haW4gKGphbS5jOjM4MykKPT0zMDcyMT09ICAg
IGJ5IDB4NDAyNjAxNEY6ICh3aXRoaW4gL2xpYi9saWJjLTIuMi41LnNvKQo9PTMwNzIxPT0g
ICAgYnkgMHg4MDQ5MDExOiAod2l0aGluIC90bXAvYm9vc3QvdG9vbHMvYnVpbGQvamFtX3Ny
Yy9iaW4ubGludXh4ODYvYmphbSkKPT0zMDcyMT09IAo9PTMwNzIxPT0gMTc0OTg0IGJ5dGVz
IGluIDE0NTgyIGJsb2NrcyBhcmUgZGVmaW5pdGVseSBsb3N0IGluIGxvc3MgcmVjb3JkIDIw
IG9mIDI1Cj09MzA3MjE9PSAgICBhdCAweDQwMDQzQkRCOiBtYWxsb2MgKHZnX2NsaWVudGZ1
bmNzLmM6MTAwKQo9PTMwNzIxPT0gICAgYnkgMHg4MDRFMDJBOiBhZGRzZXR0aW5ncyAocnVs
ZXMuYzoyMzUpCj09MzA3MjE9PSAgICBieSAweDgwNEEyQUY6IGNvbGxlY3RfYXJndW1lbnRz
IChjb21waWxlLmM6NjUyKQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBNzU5OiBldmFsdWF0ZV9y
dWxlIChjb21waWxlLmM6ODY0KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBMDcyOiBjb21waWxl
X3J1bGUgKGNvbXBpbGUuYzo1NjMpCj09MzA3MjE9PSAgICBieSAweDgwNEREQkM6IHBhcnNl
X2V2YWx1YXRlIChwYXJzZS5jOjEzOCkKPT0zMDcyMT09ICAgIGJ5IDB4ODA0OTdGRDogY29t
cGlsZV9hcHBlbmQgKGNvbXBpbGUuYzoxMjQpCj09MzA3MjE9PSAKPT0zMDcyMT09IDUxNTg1
OCBieXRlcyBpbiA3MjcyIGJsb2NrcyBhcmUgZGVmaW5pdGVseSBsb3N0IGluIGxvc3MgcmVj
b3JkIDI0IG9mIDI1Cj09MzA3MjE9PSAgICBhdCAweDQwMDQzQkRCOiBtYWxsb2MgKHZnX2Ns
aWVudGZ1bmNzLmM6MTAwKQo9PTMwNzIxPT0gICAgYnkgMHg4MDREQTJCOiBuZXdzdHIgKG5l
d3N0ci5jOjUzKQo9PTMwNzIxPT0gICAgYnkgMHg4MDRCM0QyOiB2YXJfZXhwYW5kIChleHBh
bmQuYzo0MTEpCj09MzA3MjE9PSAgICBieSAweDgwNEFGMDM6IHZhcl9leHBhbmQgKGV4cGFu
ZC5jOjIwNikKPT0zMDcyMT09ICAgIGJ5IDB4ODA0QUYwMzogdmFyX2V4cGFuZCAoZXhwYW5k
LmM6MjA2KQo9PTMwNzIxPT0gICAgYnkgMHg4MDRBRjAzOiB2YXJfZXhwYW5kIChleHBhbmQu
YzoyMDYpCj09MzA3MjE9PSAgICBieSAweDgwNDlFMjY6IGNvbXBpbGVfbGlzdCAoY29tcGls
ZS5jOjQyNykKPT0zMDcyMT09IAo9PTMwNzIxPT0gTEVBSyBTVU1NQVJZOgo9PTMwNzIxPT0g
ICAgZGVmaW5pdGVseSBsb3N0OiA3MTgyNzAgYnl0ZXMgaW4gMjMyMzAgYmxvY2tzLgo9PTMw
NzIxPT0gICAgcG9zc2libHkgbG9zdDogICAxMTkzIGJ5dGVzIGluIDE2IGJsb2Nrcy4KPT0z
MDcyMT09ICAgIHN0aWxsIHJlYWNoYWJsZTogMTk1MjkzOSBieXRlcyBpbiAxMDk3NjAgYmxv
Y2tzLgo9PTMwNzIxPT0gUmVhY2hhYmxlIGJsb2NrcyAodGhvc2UgdG8gd2hpY2ggYSBwb2lu
dGVyIHdhcyBmb3VuZCkgYXJlIG5vdCBzaG93bi4KPT0zMDcyMT09IFRvIHNlZSB0aGVtLCBy
ZXJ1biB3aXRoOiAtLXNob3ctcmVhY2hhYmxlPXllcwo9PTMwNzIxPT0gCg==
 --------------Boundary-00=_R3EOX6YH2S23RKK903L9--