CODEWITHSUNDEEP
c++operator-overloadingnew-operatordelete-operator
Brammie
Brammie

Reputation: 610

How to obtain the source location in an overload of operator new

I'm making a little memory leak finder in my program, but my way of overloading new and delete (and also new[] and delete[]) doesn't seem to do anything.

void* operator new (unsigned int size, const char* filename, int line)
{
    void* ptr = new void[size];
    memleakfinder.AddTrack(ptr,size,filename,line);
    return ptr;
}

The way I overloaded new is shown in the code snippet above. I guess it's something with the operator returning void* but I do not know what to do about it.

Upvotes: 29

Views: 87786

Answers (4)

kyku
kyku

Reputation: 6042

Maybe you can do what you want with a little bit of preprocessor magic:

#include <iostream>

using namespace std;

void* operator new (size_t size, const char* filename, int line) {
    void* ptr = new char[size];
    cout << "size = " << size << " filename = " << filename << " line = " << line << endl;
    return ptr;
}

#define new new(__FILE__, __LINE__)

int main() {
    int* x = new int;
}

Upvotes: 61

T.E.D.
T.E.D.

Reputation: 44804

I think the problem here is that your new's parameter profile doesn't match that of the standard operator new, so that one isn't getting hidden (and is thus still being used).

Your parameter profiles for new and delete need to look like this:

void* operator new(size_t);
void operator delete(void*, size_t);

Upvotes: 13

dirkgently
dirkgently

Reputation: 111130

void* ptr = new void[size];

Can't do that. Fix it.

Never ever try to overload new/delete globally. Either have them in a base class and derive all your objects from this class or use a namespace or a template allocator parameter. Why, you may ask. Because in case your program is more than a single file and using STL or other libraries you are going to screw up.

Here's a distilled version of new operator from VS2005 new.cpp:

void * operator new(size_t size) _THROW1(_STD bad_alloc)
{       // try to allocate size bytes
   void *p;
   while ((p = malloc(size)) == 0)
    if (_callnewh(size) == 0)
     {       // report no memory
        static const std::bad_alloc nomem;
        _RAISE(nomem);
     }

     return (p);
}

Upvotes: 13

Beno&#238;t
Beno&#238;t

Reputation: 16994

The problem relies with the two arguments that you have added to the overloaded new operator. Try making filename and line global in some way (or member variables if you're overloading new and delete for a single class). That should work better.

Upvotes: 4

Related Questions