2013-03-10 75 views
-3

我即将对此分段错误感到厌烦。我正在写一个简单的2通汇编器。剥离中间文件以获得操作码,符号表条目等工作正常。现在我正在将机器代码缓冲区打印到控制台,但是有些问题导致了分段错误。使用cout进行打印时出现分段错误

#include"./assembler.h" 
using namespace std; 

int main(void) 
{ 
ifstream inf("pooltab"); 
int pooltab[10],pooltab_ptr=0; 

while(true) 
{ 
    int x; 
    inf>>x; 
    if(inf.eof()) break; 
    pooltab[pooltab_ptr++] = x; 
} 
for(int i=0;i<pooltab_ptr;i++) 
    cout<<pooltab[i]<<endl; 
pooltab_ptr = 0; 
inf.close(); 

inf.open("intermediate.asm"); 
// ofstream outf("machine_code"); 
    sym_tab symtab; 
literal_tab littab; 

symtab.create_tab(); 
littab.create_tab(); 

char buf[50],*token,*m_code_buf,ch; 
int loc_cntr=0,id,ltrl,a,b; 

while(true) 
{ 
    inf.getline(buf,50); 
    if(inf.eof()) break; 
    token = strtok(buf,"(), "); 
    if(token != NULL) 
    { 
     if(token[0] == 'A' && token[1] == 'D') 
     { 
      token = strtok(NULL,",)"); 
      if(token == NULL) 
      { 
       cerr<<"null token error"; 
       return -1; 
      } 
      id = atoi(token); 
      if(id == 1 || id == 2) 
      { 
       token = strtok(NULL,",() C"); 
            if(token == NULL) 
       { 
        cerr<<"null token error"; 
        return -1; 
       } 
       loc_cntr = atoi(token); 
      } 
     } 

     if(token[0] == 'I' && token[1] == 'S') 
     { 
      token = strtok(NULL,",)"); 
      if(token == NULL) 
      { 
       cerr<<"null token error"; 
       return -1; 
      } 
      id = atoi(token); 
      if(id == 10 || id == 11) 
      { 
       token = strtok(NULL,"S,() "); 
       if(token == NULL) 
       { 
        cerr<<"null token error"; 
        return -1; 
       } 
       a = atoi(token); 
       a = symtab.get_addr(a); 
       sprintf(m_code_buf,"%03d) + %02d 0 %03d\n",loc_cntr,id,a); 
       cout<<m_code_buf; 
      } 
      else if(id == 1) 
      { 
       sprintf(m_code_buf,"%03d) + %02d 0 000\n",loc_cntr,id); 
       printf("%s",m_code_buf); 
      } 
      else if(id > 1 || id < 10) 
      { 
       token = strtok(NULL,"() "); 
       if(token == NULL) 
       { 
        cerr<<"null token error"; 
        return -1; 
       } 
       a = token[0] - 48; 
       token = strtok(NULL," (,"); 
       if(token == NULL) 
       { 
        cerr<<"null token error"; 
        return -1; 
       } 
       ch = token[0]; 
       printf("%d %d %c \n",id,a,ch); 
      } 

      loc_cntr++; 
     } 
    } 
} 
inf.close(); 
// outf.close(); 

return 0; 
} 

这是我检查在中间代码文件命令语句(IS,1) (1) (S,1)的方式。错误在sprintf。现在,当我修改代码时,故障似乎从一个地方跳到另一个地方。例如,在一次我使用cout而不是printf。然后sprintf就好了,但是cout产生了错误。那么我改变了生成pooltab的方式,现在sprintf是麻烦制造者。

+0

您可能在某处导致未定义的行为。向我们展示完整的功能(以及令牌的定义)。 – 2013-03-10 12:55:13

+0

实际代码太大,但尚未完成,但确定。 – Ajinkya 2013-03-10 13:01:41

+0

char m_code_buf [256];做了一个简单的改变。非常感谢你们。 – Ajinkya 2013-03-10 13:28:52

回答

0

这可能意味着你想写更多的缓冲区可以采取。它会让你在某人的其他记忆上写(segfault)。

2

我猜m_code_buf要么没有初始化,要么太小。

1
if(token == NULL) cerr<<"null token error"; 
a = atoi(token); 

如果tokenNULL,你刚过NULLatoi ...

+0

好吧,现在所有的错误检查都已到位,但没有运气。IS sprintf的ID = 1仍然存在问题。 – Ajinkya 2013-03-10 13:21:42

+0

fredrik有你的答案。 – nneonneo 2013-03-10 13:22:48

1

数组和指针是不一样的。

目前,您已经定义m_code_buf为指针char

char *m_code_buf; 

也就是说只是的指针。它没有指向任何地方。没有指向的char对象。你不能开始对待它,因为它实际上指向任何有效的对象。仅仅因为这个类型告诉你它将指向一个char,并不意味着它会自动执行。

当你将它更改为:

char m_code_buf[256]; 

这给你的char数组。它实际上在内存中分配256个大小的对象供您使用。这是很好的,通过这个sprintf

但是m_code_buf这里不是指针。但是,在许多上下文中,数组的名称会隐式转换为指向其第一个元素的指针。这就是为什么你可以像处理指针那样开始处理数组的名称,即使它不是。