2016-08-03 15 views
-1

我真的很困惑。我有一个C服务器,它工作得很好,但后来我添加了一些代码,我认为它工作正常,直到我的C程序随机开始将int值更改为负值。C程序是随机改变int值的

基本上我有我的服务器输出总字节发送和中途通过传输,总是在2100000000字节左右,总字节变成负值。这是我的输出文件的一个例子。如果你看我的代码,这个值不能只是负面的。所以我怀疑这是一件很奇怪的事情。

"345000","1470253912","59203","5592","2069901108" 
"348000","1470253912","475539","4194","2092449162" 
"351000","1470253912","830291","2796","2112043464" 
"354000","1470253913","243217","1398","2133985176" 
"357000","1470253913","708686","13980","-2135434834" 
"360000","1470253914","173646","9786","-2109094024" 
"363000","1470253914","514938","6990","-2089413400" 

无论如何我添加的东西,我评论它,我仍然遇到了同样的错误。仅供参考,它在我的代码中标记为“添加了新的东西”中注释掉了。 (我加了它,因为这个帖子:Terminating C program on command line, but make sure writer is finished writing

#include <stdio.h> 
    #include <stdlib.h> 
    #include <unistd.h> 
    #include <errno.h> 
    #include <string.h> 
    #include <sys/types.h> 
    #include <sys/socket.h> 
    #include <netinet/in.h> 
    #include <netdb.h> 
    #include <arpa/inet.h> 
    #include <sys/wait.h> 
    #include <signal.h> 
    #include <sys/time.h> 



int PORT_NUM = 0; 
int RecordRate = 3000; 
FILE *fp; 

typedef struct timeval timeval; 
timeval time_; 


void error(const char *msg) 
{ 
    perror(msg); 
    exit(1); 
} 

//NEW STUFF ADDED 
//void sig_handler(int signo) 
//{ 
// if (signo == SIGINT) { 
// printf("received SIGINT\n"); 
// exit(0); 
// fflush(fp); 
// } 
//} 

int main(int argc, char *argv[]) 
{ 
    int sockfd, newsockfd, portno; 
    socklen_t clilen; 
    char buffer[1000000]; 
    struct sockaddr_in serv_addr, cli_addr; 
    int n; 



    PORT_NUM = atoi(argv[1]); 
    fp = fopen(argv[2],"w"); 

//  NEW STUFF ADDED 
//  if (signal(SIGINT, sig_handler) == SIG_ERR) 
//  printf("\ncan't catch SIGINT\n"); 

    sockfd = socket(AF_INET, SOCK_STREAM, 0); 
    if (sockfd < 0) 
      error("ERROR opening socket"); 
    bzero((char *) &serv_addr, sizeof(serv_addr)); 
    //portno = atoi(argv[1]); 
    portno = PORT_NUM; 
    serv_addr.sin_family = AF_INET; 
    serv_addr.sin_addr.s_addr = INADDR_ANY; 
    serv_addr.sin_port = htons(portno); 
    if (bind(sockfd, (struct sockaddr *) &serv_addr, 
         sizeof(serv_addr)) < 0) 
         error("ERROR on binding"); 
    listen(sockfd,10); 
    clilen = sizeof(cli_addr); 



    newsockfd = accept(sockfd, 
          (struct sockaddr *) &cli_addr, 
          &clilen); 
    if (newsockfd < 0) 
       error("ERROR on accept"); 


    int counter = 0;  
    int total_bytes_sent = 0; 

    while(1){ 

    bzero(buffer,1000000); 
    n = read(newsockfd,buffer,999999); 


    if (n < 0) { 
    error("ERROR reading from socket"); 
    } 
    else if (n != 0) { 





      total_bytes_sent += n; 
      gettimeofday(&time_, NULL); 

      if(counter%RecordRate==0){ 
      printf("counter %d \n", counter); 
      printf("Bytes Sent %d \n", total_bytes_sent); 
      fprintf(fp,"\"%d\",\"%ld\",\"%d\",\"%d\",\"%d\"\n", counter, time_.tv_sec, time_.tv_usec, n,total_bytes_sent); 
      } 




      counter++; 



    } 

    } 
    fclose(fp); 
    close(newsockfd); 
    close(sockfd); 
    return 0; 
} 

我发誓我不是曳。我担心这是我添加的,但是一旦我评论出来,我得到了同样的错误。事情是在我添加代码后才出现错误。那么因果关系,相关性,没有联系?

这是怎么发生的?为什么总是在2100000000字节左右。

这不是世界末日。我的意思是我可以硬编码的东西,确保价值永远是积极的,但我很好奇这是如何发生的。谢谢。

+1

围绕'2100000000'大约'2147483647',这是最大32位'int'数字。你从这里拿走它。 –

+0

也许你需要把'int'改成'unsigned'。假设“int”是四个字节。 '2100000000'接近'0x80000000',将被认为是负值。 –

+0

Unigend只会将他的问题从发生在2位和GB位移到发生在4位和GB位。对于这种使用情况,32位是不够的,他应该使用64位数据类型。 – jsbueno

回答

6

仅仅因为普通签名int变量溢出在2 -1(2147483647)并成为负值。

由于这个数量级不足以跟踪您需要的值,因此您应该使用一个64位变量 - 将您的变量声明为long long - 并在输出的地方适当注意此变化这个值给一个文本流,你应该很好。

+0

哎呀。大声笑。谢谢 –

+2

从技术上讲,当有符号整数溢出时,会产生未定义的行为。然而,未定义行为的最常见表现形式确实是最负数(-2147483648)的概括,然后从那里继续。尽管如此,假设这将是不可避免的。 –

+1

@Jonathan:但我认为,假设这是发生在这里的(相当)安全。 –