Re: [PATCH] utils/mtabatch.c

Hi,

You are absolutely right. Please disregard. 

Yesterday I synced from cvs, and  in send_message there was: 

Msg *part;

which was never initialized, and then 

deliver_to_bearerbox(part)

Sounds familiar?
Now that I updated from cvs again, I can't see that code anymore. Seems you changed it on 30/4/2009.

I don't know how I ended up yesterday with this outdated half baked copy. *Sigh* I guess it can happen to anyone.

BR,
Nikos
  ----- Original Message ----- 
  From: Alexander Malysh 
  To: Nikos Balkanas 
  Cc: [email protected] Devel 
  Sent: Wednesday, June 10, 2009 10:24 AM
  Subject: Re: [PATCH] utils/mtabatch.c


  Hi Nikos,


  -1 for this patch. 
  1) you don't want to send each part because bearerbox will split long message for you and ensure that it goes via the same SMSC
  2) I don't see where mtbatch should core dump, please clarify.


  Thanks,
  Alex


  Am 09.06.2009 um 20:07 schrieb Nikos Balkanas:


    Hi,

    This is a simple patch to mtbatch, that if left as is should core dump.

    BR,
    Nikos<mtbatch.diff>