< Previous by Date Date Index Next by Date >
  Thread Index  

[reSIProcate] Should this work?


I noticed a little oddity, and perhaps it's a compiler particular but in
HeaderFieldValue.hxx we declare:

      const char* mField;

And then constructor:

   mField = new char[mFieldLength];
   memcpy(const_cast<char*>(mField), hfv.mField, mFieldLength); // <- good
reason for casting

BUT in the destructor

     delete[] mField; // <- should this work?

Which I think should be something like:

    delete[]  const_cast<char*>(mField);

No?



Thomas Gal
Lead Protocol Engineer

thomasgal@xxxxxxxxxxxx
Toll Free: 877-977-0707 Say "Tom Gal"
Phone: 858-707-0707x135
Fax: 858-707-7072

LumenVox LLC 
3615 Kearny Villa Rd #202
San Diego, CA 92123
www.lumenVox.com

Attachment: smime.p7s
Description: S/MIME cryptographic signature