2016-08-03 3 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" 

В любом случае, что я добавил, я прокомментировал это, и я до сих пор сталкивался с той же ошибкой. Просто для справки он прокомментирован в моем коде под тегами «NEW STUFF ADDED». (Я добавил его из-за этот пост: 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. 32bit просто недостаточно для этого варианта использования, и он должен использовать 64-битный тип данных. – jsbueno

ответ

6

Просто потому, что обычные подписи int переполнения переменных при 2 -1 (2147483647) и становятся отрицательными.

Поскольку этого порядка величины недостаточно, чтобы отслеживать нужные вам значения, вы должны использовать для этого 64-битную переменную - объявите свою переменную как long long - и должным образом позаботьтесь об этом изменении в местах, которые вы выводите это значение в текстовый поток, и вы должны быть хорошими.

+0

О, господа. лол. Спасибо –

+2

Технически, когда знаковое целочисленное переполнение, вы несете неопределенное поведение. Однако наиболее распространенное проявление неопределенного поведения действительно является обманом для наиболее отрицательного числа (-2147483648), а затем исходя из этого. Небезопасно предположить, что это произойдет. –

+1

@Jonathan: но (довольно) безопасно предположить, что это происходит здесь, я думаю. –

Смежные вопросы