Attached is the new patch with the slight change I figured for now the easiest was to just move the FreeConfig for the smtpItems to the end of freeall:, well maybe we want to think about a struct for these items that can be global instead of the listnodes, *shrug* let me know.
Thanks, Leif p.s. I belive after a lopng bout of checks with valgrind ( an amazing package for many reasons) that the memory leaks if any are only in the main function of pop3, as far as I can tell imapd is clean! :) p.p.s. I have started on server side sorting code, may have something by this weekend I can share. has ISCS thought about or started any similar code before I get to far into it? thx. > All, > > I found one ting this breaks I will send a patch shortly.. Bascialy > bounce.c uses the config as a global. I don't see that this is a great > idea? I will be moving: > field_t dbmail_from_address, sendmail, postmaster; > > to be global instead, so I can still FreeConfig in the main function. > > Thanks, > Leif > > >> Cool, >> >> Ilja, I have attached a patch that adds a FreeConfig function, this >> doesn't solve my memory leaks from list.c addnode, but makes it a lot >> easier to debug as it free's the config list's right after their used >> instead of at the end of the main (). Please commit it to cvs if you >> feel >> it warrants it. >> >> Thanks, >> leif >> >> >>> I've put the patch in CVS, because we'll need it anyway. This way, it's >>> also easier to debug it, because a simple cvs update will get everybody >>> the new code :) >>> >>> The delivery chain is still buggy: When delivering messages through >>> LMTP, all messages get delivered twice. >>> >>> Ilja >>> >>> Leif Jackson wrote: >>> >>>> Ilja & Aaron, >>>> >>>> I am looking for this patch. I cannot access it from sourceforge? Or >>>> do >>>> I >>>> have to checkout from cvs differently than the default dbmail root? I >>>> would be glad to look at this.... Also Aaron, i was working on your >>>> sieve >>>> code but there are some issues between the current libsieve you have >>>> in >>>> cvs and the last one posed on sourceforge, the api and the lib doesn't >>>> match quite right or at least not to the code you have in the dbmail >>>> cvs >>>> tree, i am really exited about this feature and would like to help. >>>> >>>> Thanks, >>>> Leif >>>> >>>>
dbmail-2.0cvs030304-list.c.diff
Description: Binary data