Read() stuck on trying to read file












0















I have a checkpoint file that receives a server state. This states represents serialized commands that pass trough my network.



I'm trying to read the file but the it gets stuck on the read while loop.



My read function:



struct message_t *pmanager_readop(int fd){
if (fd < 0) return NULL;

// Variables
char *buffer = NULL;
int result, msg_size;
struct message_t *msg;

// Check if file has data
lseek (fd, 0, SEEK_END);
int size_ckp = lseek(fd, 0, SEEK_CUR);

if (size_ckp <= 0)
return NULL;

// Read message size
result = read_all(fd, (char *) &msg_size, 4);
if (result < 0) {
return NULL;
}
msg_size = ntohl(msg_size);
// ............


My read_all() function:



int read_all(int sock, char *buf, int len){
int bufsize = len;
while(len > 0){
int res = read(sock,buf,len);

if(res < 0){
if (errno == EINTR) continue;
return res;
}
buf += res;
len -= res;
}
return bufsize;
}


I use this same function to read data from my server/client connection with the same serialization and format but with a socket descriptor, and it works perfectly.










share|improve this question


















  • 1





    Also read() returns ssize_t not int.

    – alk
    Nov 25 '18 at 16:21






  • 1





    And lseek() returns off_t, not int.

    – Andrew Henle
    Nov 25 '18 at 16:35
















0















I have a checkpoint file that receives a server state. This states represents serialized commands that pass trough my network.



I'm trying to read the file but the it gets stuck on the read while loop.



My read function:



struct message_t *pmanager_readop(int fd){
if (fd < 0) return NULL;

// Variables
char *buffer = NULL;
int result, msg_size;
struct message_t *msg;

// Check if file has data
lseek (fd, 0, SEEK_END);
int size_ckp = lseek(fd, 0, SEEK_CUR);

if (size_ckp <= 0)
return NULL;

// Read message size
result = read_all(fd, (char *) &msg_size, 4);
if (result < 0) {
return NULL;
}
msg_size = ntohl(msg_size);
// ............


My read_all() function:



int read_all(int sock, char *buf, int len){
int bufsize = len;
while(len > 0){
int res = read(sock,buf,len);

if(res < 0){
if (errno == EINTR) continue;
return res;
}
buf += res;
len -= res;
}
return bufsize;
}


I use this same function to read data from my server/client connection with the same serialization and format but with a socket descriptor, and it works perfectly.










share|improve this question


















  • 1





    Also read() returns ssize_t not int.

    – alk
    Nov 25 '18 at 16:21






  • 1





    And lseek() returns off_t, not int.

    – Andrew Henle
    Nov 25 '18 at 16:35














0












0








0








I have a checkpoint file that receives a server state. This states represents serialized commands that pass trough my network.



I'm trying to read the file but the it gets stuck on the read while loop.



My read function:



struct message_t *pmanager_readop(int fd){
if (fd < 0) return NULL;

// Variables
char *buffer = NULL;
int result, msg_size;
struct message_t *msg;

// Check if file has data
lseek (fd, 0, SEEK_END);
int size_ckp = lseek(fd, 0, SEEK_CUR);

if (size_ckp <= 0)
return NULL;

// Read message size
result = read_all(fd, (char *) &msg_size, 4);
if (result < 0) {
return NULL;
}
msg_size = ntohl(msg_size);
// ............


My read_all() function:



int read_all(int sock, char *buf, int len){
int bufsize = len;
while(len > 0){
int res = read(sock,buf,len);

if(res < 0){
if (errno == EINTR) continue;
return res;
}
buf += res;
len -= res;
}
return bufsize;
}


I use this same function to read data from my server/client connection with the same serialization and format but with a socket descriptor, and it works perfectly.










share|improve this question














I have a checkpoint file that receives a server state. This states represents serialized commands that pass trough my network.



I'm trying to read the file but the it gets stuck on the read while loop.



My read function:



struct message_t *pmanager_readop(int fd){
if (fd < 0) return NULL;

// Variables
char *buffer = NULL;
int result, msg_size;
struct message_t *msg;

// Check if file has data
lseek (fd, 0, SEEK_END);
int size_ckp = lseek(fd, 0, SEEK_CUR);

if (size_ckp <= 0)
return NULL;

// Read message size
result = read_all(fd, (char *) &msg_size, 4);
if (result < 0) {
return NULL;
}
msg_size = ntohl(msg_size);
// ............


My read_all() function:



int read_all(int sock, char *buf, int len){
int bufsize = len;
while(len > 0){
int res = read(sock,buf,len);

if(res < 0){
if (errno == EINTR) continue;
return res;
}
buf += res;
len -= res;
}
return bufsize;
}


I use this same function to read data from my server/client connection with the same serialization and format but with a socket descriptor, and it works perfectly.







c file io






share|improve this question













share|improve this question











share|improve this question




share|improve this question










asked Nov 25 '18 at 16:14









Filipe NóbregaFilipe Nóbrega

11318




11318








  • 1





    Also read() returns ssize_t not int.

    – alk
    Nov 25 '18 at 16:21






  • 1





    And lseek() returns off_t, not int.

    – Andrew Henle
    Nov 25 '18 at 16:35














  • 1





    Also read() returns ssize_t not int.

    – alk
    Nov 25 '18 at 16:21






  • 1





    And lseek() returns off_t, not int.

    – Andrew Henle
    Nov 25 '18 at 16:35








1




1





Also read() returns ssize_t not int.

– alk
Nov 25 '18 at 16:21





Also read() returns ssize_t not int.

– alk
Nov 25 '18 at 16:21




1




1





And lseek() returns off_t, not int.

– Andrew Henle
Nov 25 '18 at 16:35





And lseek() returns off_t, not int.

– Andrew Henle
Nov 25 '18 at 16:35












1 Answer
1






active

oldest

votes


















1














You ought to handle the case that read() returns 0, telling you that the other side shut-down the connection if reading from a socket descriptor, or EOF encountered if reading from a file descriptor.






share|improve this answer

























    Your Answer






    StackExchange.ifUsing("editor", function () {
    StackExchange.using("externalEditor", function () {
    StackExchange.using("snippets", function () {
    StackExchange.snippets.init();
    });
    });
    }, "code-snippets");

    StackExchange.ready(function() {
    var channelOptions = {
    tags: "".split(" "),
    id: "1"
    };
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function() {
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled) {
    StackExchange.using("snippets", function() {
    createEditor();
    });
    }
    else {
    createEditor();
    }
    });

    function createEditor() {
    StackExchange.prepareEditor({
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader: {
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    },
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    });


    }
    });














    draft saved

    draft discarded


















    StackExchange.ready(
    function () {
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53469390%2fread-stuck-on-trying-to-read-file%23new-answer', 'question_page');
    }
    );

    Post as a guest















    Required, but never shown

























    1 Answer
    1






    active

    oldest

    votes








    1 Answer
    1






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    1














    You ought to handle the case that read() returns 0, telling you that the other side shut-down the connection if reading from a socket descriptor, or EOF encountered if reading from a file descriptor.






    share|improve this answer






























      1














      You ought to handle the case that read() returns 0, telling you that the other side shut-down the connection if reading from a socket descriptor, or EOF encountered if reading from a file descriptor.






      share|improve this answer




























        1












        1








        1







        You ought to handle the case that read() returns 0, telling you that the other side shut-down the connection if reading from a socket descriptor, or EOF encountered if reading from a file descriptor.






        share|improve this answer















        You ought to handle the case that read() returns 0, telling you that the other side shut-down the connection if reading from a socket descriptor, or EOF encountered if reading from a file descriptor.







        share|improve this answer














        share|improve this answer



        share|improve this answer








        edited Nov 26 '18 at 7:20

























        answered Nov 25 '18 at 16:22









        alkalk

        58.6k763172




        58.6k763172






























            draft saved

            draft discarded




















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function () {
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53469390%2fread-stuck-on-trying-to-read-file%23new-answer', 'question_page');
            }
            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Lallio

            Futebolista

            Jornalista