Crashing while deleting

#include <vector>
// Your way
void do_something_1()
{
    licenseClientData temp;
    temp.clientData = new licenseClient*[3]; // pointer to array of pointers to  licenseClient
    temp.iCount = 0;

    // ….

    temp.iCount--;
    delete temp.clientData[temp.iCount];
    temp.iCount--;
    delete temp.clientData[temp.iCount];
    temp.iCount--;
    delete temp.clientData[temp.iCount];

    delete []  temp.clientData; // remember to delete the array as well
    getch();
    return 0;
}
// Better way
void do_something_2()
{
    std::vector<SOCKET> licenseClient(3,(SOCKET()));
    // do something with array
}
// Another way
void do_something_3()
{
    std::vector<SOCKET> licenseClient;
    licenseClient.push_back((SOCKET()));
    licenseClient.push_back((SOCKET()));
    licenseClient.push_back((SOCKET()));
    // do something with array
}
// Or even
void do_something_4()
{
     SOCKET licenseClient[3] = {SOCKET(), SOCKET(), SOCKET()};
    // do something with array
}

#include <vector>

struct licenseClient
{
    SOCKET  socketNo;
};

struct licenseClientData
{
    // Not required anymore uses clientData.size() instead.
    // int             iCount;

    std::vector<licenseClient > clientData;
};

int _tmain(int argc, _TCHAR* argv[])
{
    licenseClientData temp;

    licenseClient lc1;
    temp.clientData.push_back(lc1);

    licenseClient lc2;
    temp.clientData.push_back(lc2);

    licenseClient lc3;
    temp.clientData.push_back(lc3);

    // The following is not necessary if the data 
    // is not used elsewhere in the real program.
    temp.clientData.erase(temp.clientData.back());
    temp.clientData.erase(temp.clientData.back());
    temp.clientData.erase(temp.clientData.back());

    getch();
    return 0;
}

if (temp.clientData.size() > 0)
{
    temp.clientData.erase(temp.clientData.back());
}