Styjun

I've created this small Player Management System. But getting this error whenever I search or update information of a player. Also, I'm not doing any dynamic memory allocation, so I am not sure why there is a problem of freeing the pointer.

a.out(1599,0x10802e5c0) malloc: *** error for object 0x7fd4d0d000c0: pointer being freed was not allocated
a.out(1599,0x10802e5c0) malloc: *** set a breakpoint in malloc_error_break to debug
Abort trap: 6

Segmentation fault: 11 is also printed out.

Player.cc

#include <iostream>
#include <string>
#include <fstream>

using namespace std;

fstream f;

class Player

public:
 string name;
 string dob;
 string bowling_skill;
 string batting_hand;
 string country;
 string team;
 int runs;
 int fours;
 int sixes;

 void info()
 
 cout << "Name: " << this->name << "n";
 cout << "Date of Birth: " << this->dob << "n";
 cout << "Bowling Skill: " << this->bowling_skill << "n";
 cout << "Batting hand: : " << this->batting_hand << "n";
 cout << "Country: " << this->country << "n";
 cout << "Team: " << this->team << "n";
 cout << "Runs: " << this->runs << "n";
 cout << "No. of fours: " << this->fours << "n";
 cout << "No. of sixes: " << this->sixes << "nn";
 
;

void searchPlayer(string name)

 Player player;
 int found = 0;

 f.open("Database/Player.dat", ios::in 

void addPlayer()
 ios::binary 

void updatePlayer(string name)
 ios::binary 

Main.cc

#include <iostream>
#include "Classes/Player.cc"

using namespace std;

void playerMenu()

 while (1)
 
 string input;
 int option;

 cout << "1 Searchn";
 cout << "2 Addn";
 cout << "3 Updaten";
 cout << "4 Deleten";
 cout << "5 Back to main menunn";

 cout << "OPTION: ";
 cin >> option;
 system("clear");

 switch (option)
 
 case 1:
 cout << "Enter name of the player: ";
 cin.ignore(100, 'n');
 getline(cin, input);
 cout << "n";
 searchPlayer(input);
 break;
 case 2:
 addPlayer();
 break;
 case 3:
 cout << "Enter name of the player: ";
 cin.ignore(100, 'n');
 getline(cin, input);
 updatePlayer(input);
 break;
 case 5:
 return;
 default:
 cout << "Please choose a valid optionn";
 
 


int main()

 while (1)
 
 int option;

 cout << "1 Playernn";

 cout << "OPTION: ";
 cin >> option;
 system("clear");

 switch (option)
 
 case 1:
 playerMenu();
 break;
 default:
 cout << "nPlease choose a valid optionn";
 
 

Any help you can provide is appreciable. Thanks

You're writing and reading your Player objects directly as their binary representation, but they contain non-POD data such as std::string, which contains pointers inside. This is a guaranteed ticket to UndefinedBehaviour-land.

You must change your input/output routines so that they serialise the Player object in some sensible way, such as storing the length of each string followed by its contents, and reading appropriately.