Fix typo in comment.
[cascardo/rnetclient.git] / rnetclient.c
index 0ba06a8..946fa79 100644 (file)
@@ -72,6 +72,9 @@ struct rnetclient_args {
 
        /* Output directory to save the receipt.  */
        char *output_dir;
+
+       /* Output filename. */
+       char output_file[PATH_MAX];
 };
 
 /* Parser for command line arguments.  */
@@ -277,7 +280,7 @@ static int rnet_send(gnutls_session_t session, char *buffer, size_t len, int hea
           The uploaded file may be larger: to upload such large
           files, it suffices to send multiple records till the entire
           file is transferred, without waiting for a response.  Since
-          we've alread informed the server of the file size in the
+          we've already informed the server of the file size in the
           header, it knows exactly how much data to expect before
           sending a response.  It will only send an error message
           before that if it times us out.
@@ -343,30 +346,13 @@ static int rnet_recv(gnutls_session_t session, struct rnet_message **message)
        return 0;
 }
 
-static void save_rec_file(char *cpf, char *buffer, int len, const struct rnetclient_args *args)
+static int open_rec_file(char *cpf, struct rnetclient_args *args)
 {
        int fd;
-       char cwd[PATH_MAX];
-       char *path, *fname, *tmp;
-       size_t fname_len;
-       ssize_t r;
-       /* If the user provided the output directory where she wishes
-          to save the receipt, then we use it.  Otherwise, we save
-          the file in the current working directory (CWD).  */
-       if (args->output_dir == NULL)
-               path = getcwd(cwd, PATH_MAX);
-       else {
-               struct stat st;
-               if (stat(args->output_dir, &st) < 0) {
-                       fprintf(stderr, "Could not stat directory \"%s\": %s\n", args->output_dir, strerror(errno));
-                       return;
-               }
-               if (!S_ISDIR(st.st_mode)) {
-                       fprintf(stderr, "Error: \"%s\" is a not a directory.\n", args->output_dir);
-                       return;
-               }
-               path = args->output_dir;
-       }
+       char *path, *tmp;
+
+       path = args->output_dir;
+
        /* Now it's time to decide which filename to write.  We use
           the declaration's filename as a base layout, because the
           proprietary version of the IRPF program only recognizes
@@ -386,47 +372,53 @@ static void save_rec_file(char *cpf, char *buffer, int len, const struct rnetcli
                   just have to change one letter.  */
                tmp = strstr(p, ".DEC");
                tmp[1] = 'R';
-               fname_len = strlen(p) + strlen(path) + 2;
-               fname = alloca(fname_len);
-               snprintf(fname, fname_len, "%s/%s", path, p);
+               snprintf(args->output_file, PATH_MAX, "%s/%s", path, p);
                free(p);
        } else {
                /* The declaration filename does not follow the
                   convention, so we will not use it as a template.
                   We just generate a filename using "$CPF.REC".  */
-               fname_len = strlen(cpf) + strlen(path) + sizeof(".REC") + 2;
-               fname = alloca(fname_len);
-               snprintf(fname, fname_len, "%s/%s.REC", path, cpf);
+               snprintf(args->output_file, PATH_MAX, "%s/%s.REC", path, cpf);
        }
        /* Now, open the file and write.  */
-       fd = open(fname, O_CREAT | O_WRONLY | O_EXCL, S_IRUSR | S_IWUSR);
+       fd = open(args->output_file, O_CREAT | O_WRONLY | O_EXCL, S_IRUSR | S_IWUSR);
        if (fd < 0) {
-               fprintf(stderr, "Could not create receipt file \"%s\": %s\n", fname, strerror(errno));
-               return;
+               fprintf(stderr, "Could not create receipt file \"%s\": %s\n",
+                       args->output_file, strerror(errno));
        }
+       return fd;
+}
+
+static int save_rec_file(int fd, char *buffer, int len, const struct rnetclient_args *args)
+{
+       ssize_t r;
+
        do {
                r = write(fd, buffer, len);
        } while (r < 0 && errno == EAGAIN);
-       if (r != len)
-               fprintf(stderr, "Could not write to receipt file: %s", strerror(errno));
-       else
-               fprintf(stderr, "Wrote the receipt file to %s.\n", fname);
-       close(fd);
+       if (r != len) {
+               fprintf(stderr, "Could not write to receipt file: %s",
+                       strerror(errno));
+       } else {
+               fprintf(stderr, "Wrote the receipt file to %s.\n",
+                       args->output_file);
+       }
+       return r;
 }
 
-static void handle_response_text_and_file(char *cpf, struct rnet_message *message, const struct rnetclient_args *args)
+static void handle_response_text_and_file(int fd, struct rnet_message *message, const struct rnetclient_args *args)
 {
        char *value;
        int vlen;
        if (!rnet_message_parse(message, "texto", &value, &vlen))
                fprintf(stderr, "%.*s\n", vlen, value);
        if (!rnet_message_parse(message, "arquivo", &value, &vlen))
-               save_rec_file(cpf, value, vlen, args);
+               save_rec_file(fd, value, vlen, args);
 }
 
-static void handle_response_already_found(char *cpf, struct rnet_message *message, const struct rnetclient_args *args)
+static void handle_response_already_found(int fd, struct rnet_message *message, const struct rnetclient_args *args)
 {
-       handle_response_text_and_file(cpf, message, args);
+       handle_response_text_and_file(fd, message, args);
 }
 
 static void handle_response_error(struct rnet_message *message)
@@ -441,7 +433,7 @@ static void handle_response_error(struct rnet_message *message)
 int main(int argc, char **argv)
 {
        int c;
-       int r;
+       int r = 0;
        struct rnet_decfile *decfile;
        struct rnet_message *message = NULL;
        struct rnetclient_args rnet_args;
@@ -449,6 +441,8 @@ int main(int argc, char **argv)
        int finish = 0;
        char *cpf;
        error_t err;
+       char cwd[PATH_MAX];
+       int outfd;
 
        /* Parsing the command line arguments.  The argp_parse
           function calls exit() if there is some error during the
@@ -463,6 +457,25 @@ int main(int argc, char **argv)
        if (err != 0)
                fprintf(stderr, "internal error while parsing command line arguments.");
 
+       /* If the user provided the output directory where she wishes
+          to save the receipt, then we use it.  Otherwise, we save
+          the file in the current working directory (CWD).  */
+       if (rnet_args.output_dir == NULL) {
+               rnet_args.output_dir = getcwd(cwd, PATH_MAX);
+       } else {
+               struct stat st;
+               if (stat(rnet_args.output_dir, &st) < 0) {
+                       fprintf(stderr, "Could not stat directory \"%s\": %s\n",
+                               rnet_args.output_dir, strerror(errno));
+                       exit(1);
+               }
+               if (!S_ISDIR(st.st_mode)) {
+                       fprintf(stderr, "Error: \"%s\" is a not a directory.\n",
+                               rnet_args.output_dir);
+                       exit(1);
+               }
+       }
+
        decfile = rnet_decfile_open(rnet_args.input_file);
        if (!decfile) {
                fprintf(stderr, "could not parse file \"%s\": %s\n", rnet_args.input_file, strerror(errno));
@@ -471,6 +484,12 @@ int main(int argc, char **argv)
 
        cpf = rnet_decfile_get_header_field(decfile, "cpf");
 
+       outfd = open_rec_file(cpf, &rnet_args);
+       if (outfd < 0) {
+               r = 1;
+               goto out_rec;
+       }
+
        gnutls_global_init();
 
        session_new(&session);
@@ -478,20 +497,26 @@ int main(int argc, char **argv)
        if (r) {
                fprintf(stderr, "error connecting to server: %s\n",
                        r == EAI_SYSTEM ? strerror(errno) : gai_strerror(r));
-               exit(1);
+               r = 1;
+               goto out_connect;
        }
        gnutls_transport_set_ptr(session, (gnutls_transport_ptr_t)(intptr_t) c);
        r = handshake(c);
        if (r < 0) {
-               exit(1);
+               r = 1;
+               goto out_handshake;
        }
-       if ((r = gnutls_handshake(session)) < 0)
+       if ((r = gnutls_handshake(session)) < 0) {
                fprintf(stderr, "error in handshake: %s\n",
                                gnutls_strerror(r));
+               r = 1;
+               goto out_handshake;
+       }
 
        r = rnet_encode(decfile, &message);
        if (r < 0) {
                fprintf(stderr, "error encoding message, file not supported?\n");
+               r = 1;
                goto out;
        }
 
@@ -502,23 +527,24 @@ int main(int argc, char **argv)
        r = rnet_recv(session, &message);
        if (r || !message || message->len == 0) {
                fprintf(stderr, "error when receiving response\n");
+               r = 1;
                goto out;
        }
        switch (message->buffer[0]) {
        case 1: /* go ahead */
-               handle_response_text_and_file(cpf, message, &rnet_args);
+               handle_response_text_and_file(outfd, message, &rnet_args);
                break;
        case 3: /* error */
                handle_response_error(message);
                finish = 1;
                break;
        case 4:
-               handle_response_already_found(cpf, message, &rnet_args);
+               handle_response_already_found(outfd, message, &rnet_args);
                finish = 1;
                break;
        case 2:
        case 5:
-               handle_response_text_and_file(cpf, message, &rnet_args);
+               handle_response_text_and_file(outfd, message, &rnet_args);
                finish = 1;
                break;
        }
@@ -534,6 +560,7 @@ int main(int argc, char **argv)
        r = rnet_recv(session, &message);
        if (r || !message || message->len == 0) {
                fprintf(stderr, "error when receiving response\n");
+               r = 1;
                goto out;
        }
        switch (message->buffer[0]) {
@@ -544,15 +571,19 @@ int main(int argc, char **argv)
        case 4:
        case 5:
        case 1:
-               handle_response_text_and_file(cpf, message, &rnet_args);
+               handle_response_text_and_file(outfd, message, &rnet_args);
                break;
        }
        
 out:
        gnutls_bye(session, GNUTLS_SHUT_RDWR);
+out_handshake:
        close(c);
-       rnet_decfile_close(decfile);
+out_connect:
        gnutls_global_deinit();
+       close(outfd);
+out_rec:
+       rnet_decfile_close(decfile);
 
-       return 0;
+       return r;
 }