Another coredump issue in C

166 views Asked by At

when I compile my code with
g++ -g -o prueba prueba.cpp -lstdc++ -O3 -march=corei7 -mtune=corei7 -std=c++0x

After debugging with g++ -g prueba.cpp, I got these:

prueba.cpp:24:6: error: ‘stoi’ is not a member of ‘std’
tm = std::stoi(string2);
     ^
prueba.cpp:34:7: error: ‘stoi’ is not a member of ‘std’
ler = std::stoi(string1);
      ^
prueba.cpp:77:8: error: ‘stoi’ is not a member of ‘std’
C[i]=std::stoi(string);
     ^

The way I declare stoi as std::stoi was based on this example.

And the broken block is:

//////////////////////////////////////////
if( B != NULL )
{
    for(i=0;i<div;i++)
    {
        a=B[i][0];
        b=B[i][1];
        std::cout<<a<<"\t"<<b<<"\n";
        A[b][a]=A[b][a]+1;
        A[a][b]=A[a][b]+1;
    }   
    free(B);
}
//////////////////////////////////////////

which is given me the problem with segmentation fault. But I don't see what or where is the problem.

The files are:

LER https://app.box.com/s/oi52zw7j8w19txr4cau2pf4w3pzmcelm

REL https://app.box.com/s/bo2xwm2hviucx4jzarxv9ghg11j72goa

TM https://app.box.com/s/ofmhsttqujor6di0tm89tiiikm3ou1xj

The full code is:

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <iostream>
#include <fstream>
#include <string.h>

using namespace std;

int main()
{
int a,b,div,value,k,i,j,tm,ler;  
char string[256];
char string1[256];
char string2[256];

FILE *TM = fopen("TM","r");
if(TM == NULL)
{  
    printf("Can't open %s\n","TM");
    exit(1);
}
fscanf(TM,"%255s",string2);
tm = std::stoi(string2);
fclose(TM);

FILE *LER = fopen("LER","r");
if(LER == NULL)
{  
    printf("Can't open %s\n","LER");
    exit(1);
}
fscanf(LER,"%255s",string1);
ler = std::stoi(string1);
fclose(LER);

div=ler/2;

int **A;
A = (int **)malloc(tm*sizeof(int*));
for(j=0;j<tm;j++)
{
    A[j]=(int*)malloc(tm*sizeof(int));
}

int **B;
B = (int **)malloc(div*sizeof(int*));
for(j=0;j<div;j++)
{
    B[j]=(int*)malloc(2*sizeof(int));
}

int *C;
C = (int*) malloc(ler*sizeof(int));

if( A != NULL )
{
    for(i=0;i<tm;i++)
    {
        for(j=0;j<tm;j++)
        {
            A[i][j]=0;
        }
    }
}

FILE *stream = fopen("REL","r");
if(stream == NULL)
{  
    printf("Can't open %s\n","REL");
    exit(1);
}

for(i=0;i<ler;i++)
{
        fscanf(stream,"%255s",string);
        C[i]=std::stoi(string);
}
fclose(stream);

if( C != NULL )
{
    k=0;
    for(i=0;i<div;i++)
    {
        for(j=0;j<2;j++)
        {
            B[i][j]=C[k];
            k++;
        }
    }
    free(C);
}
//////////////////////////////////////////
if( B != NULL )
{
    for(i=0;i<div;i++)
    {
        a=B[i][0];
        b=B[i][1];
        std::cout<<a<<"\t"<<b<<"\n";
        A[b][a]=A[b][a]+1;
        A[a][b]=A[a][b]+1;
    }   
    free(B);
}
//////////////////////////////////////////
for(i=0;i<tm;i++)
{
    for(j=0;j<tm;j++)
    {
        cout<<A[i][j]; 
    }
    cout<<"\n"; 
}
free(A);
}
1

There are 1 answers

1
dvhh On BEST ANSWER

I would think you would need to add check on your externally accepted values (using assert might be a start) Like :

  • checking tm>0
  • ler>0
  • C[i]<tm
  • A[i]!=NULL
  • B[i]!=NULL

As mentionned in the comment it an off by one issue :

in LER the values should be from 0 to tm-1
or use B[i][j]=C[k]-1; at line 88