Add TFTP upload spinner
[oweals/nmrpflash.git] / nmrp.c
diff --git a/nmrp.c b/nmrp.c
index f6a1e032c0fb6270f5113e57573c2dadc8c090d4..9df5b307a09eb5db0184a774ea6cce50bfddd97f 100644 (file)
--- a/nmrp.c
+++ b/nmrp.c
 #include "nmrpd.h"
 
 #define NMRP_HDR_LEN 6
-#define NMRP_OPT_LEN 4
+#define NMRP_OPT_HDR_LEN 4
 #define NMRP_MIN_PKT_LEN (sizeof(struct eth_hdr) +  NMRP_HDR_LEN)
 
-#define NMRP_MAX_OPT_SIZE 12
-#define NMRP_MAX_OPT_NUM 2
-
 #define ETH_P_NMRP 0x0912
-#define IP_LEN 4
-#define MAX_LOOP_RECV 1024
 
 #ifndef PACKED
 #define PACKED __attribute__((__packed__))
 #endif
 
+#ifdef NMRPFLASH_WINDOWS
+#define setenv(name, value, overwrite) SetEnvironmentVariable(name, value)
+#endif
+
 enum nmrp_code {
        NMRP_C_NONE = 0,
        NMRP_C_ADVERTISE = 1,
@@ -65,13 +64,7 @@ enum nmrp_opt_type {
 struct nmrp_opt {
        uint16_t type;
        uint16_t len;
-       union {
-               uint8_t magic[4];
-               struct {
-                       uint8_t addr[4];
-                       uint8_t mask[4];
-               } ip;
-       } val;
+       char val[1];
 } PACKED;
 
 struct nmrp_msg {
@@ -79,14 +72,7 @@ struct nmrp_msg {
        uint8_t code;
        uint8_t id;
        uint16_t len;
-       struct nmrp_opt opts[2];
-       uint32_t num_opts;
-} PACKED;
-
-struct eth_hdr {
-       uint8_t ether_dhost[6];
-       uint8_t ether_shost[6];
-       uint16_t ether_type;
+       char opts[44];
 } PACKED;
 
 struct nmrp_pkt {
@@ -94,110 +80,185 @@ struct nmrp_pkt {
        struct nmrp_msg msg;
 } PACKED;
 
-static void msg_update_len(struct nmrp_msg *msg)
+static const char *msg_code_str(uint16_t code)
 {
-       uint32_t i = 0;
-       msg->len = NMRP_HDR_LEN;
-       for (; i != msg->num_opts; ++i) {
-               msg->len += msg->opts[i].len;
+#define MSG_CODE(x) case NMRP_C_ ## x: return #x
+       static char buf[16];
+
+       switch (code) {
+               MSG_CODE(ADVERTISE);
+               MSG_CODE(CONF_REQ);
+               MSG_CODE(CONF_ACK);
+               MSG_CODE(CLOSE_REQ);
+               MSG_CODE(CLOSE_ACK);
+               MSG_CODE(KEEP_ALIVE_REQ);
+               MSG_CODE(KEEP_ALIVE_ACK);
+               MSG_CODE(TFTP_UL_REQ);
+               default:
+                       snprintf(buf, sizeof(buf), "%04x", ntohs(code));
+                       return buf;
        }
+#undef MSG_CODE
+}
+
+static uint16_t to_region_code(const char *region)
+{
+#define REGION_CODE(r, c) if (!strcasecmp(region, r)) return htons(c)
+       REGION_CODE("NA", 0x0001);
+       REGION_CODE("WW", 0x0002);
+       REGION_CODE("GR", 0x0003);
+       REGION_CODE("PR", 0x0004);
+       REGION_CODE("RU", 0x0005);
+       REGION_CODE("BZ", 0x0006);
+       REGION_CODE("IN", 0x0007);
+       REGION_CODE("KO", 0x0008);
+       REGION_CODE("JP", 0x0009);
+#undef REGION_CODE
+       return 0;
 }
 
-static void msg_dump(struct nmrp_msg *msg, int dump_opts)
+static void msg_dump(struct nmrp_msg *msg)
 {
-       struct nmrp_opt *opt;
-       int remain_len, len, i;
+       int rem;
 
        fprintf(stderr, "res=0x%04x, code=0x%02x, id=0x%02x, len=%u",
-                       msg->reserved, msg->code, msg->id, msg->len);
+                       ntohs(msg->reserved), msg->code, msg->id, ntohs(msg->len));
 
-       remain_len = msg->len - NMRP_HDR_LEN;
-       fprintf(stderr, "%s\n", remain_len ? "" : " (no opts)");
+       rem = ntohs(msg->len) - NMRP_HDR_LEN;
+       fprintf(stderr, "%s\n", rem ? "" : " (no opts)");
+}
 
-       if (dump_opts) {
-               opt = msg->opts;
+static void *msg_opt(struct nmrp_msg *msg, uint16_t type, uint16_t* len)
+{
+       struct nmrp_opt* opt = (struct nmrp_opt*)msg->opts;
+       size_t rem = ntohs(msg->len) - NMRP_HDR_LEN;
+       uint16_t olen;
 
-               while (remain_len > 0) {
-                       len = opt->len;
-                       fprintf(stderr, "  opt type=%u, len=%u", opt->type, len);
-                       for (i = 0; i != len - NMRP_OPT_LEN; ++i) {
-                               if (!(i % 16)) {
-                                       fprintf(stderr, "\n  ");
-                               }
+       do {
+               olen = ntohs(opt->len);
+               if (olen < NMRP_OPT_HDR_LEN || olen > rem) {
+                       break;
+               }
 
-                               fprintf(stderr, "%02x ", ((char*)&opt->val)[i] & 0xff);
+               if (ntohs(opt->type) == type) {
+                       if (len) {
+                               *len = olen;
                        }
-                       fprintf(stderr, "\n");
-                       remain_len -= len;
-                       opt = (struct nmrp_opt*)(((char*)opt) + len);
+
+                       return opt->val;
                }
-       }
-}
 
-static void msg_hton(struct nmrp_msg *msg)
-{
-       uint32_t i = 0;
+               opt = (struct nmrp_opt*)(((char *)opt) + olen);
+               rem -= olen;
+       } while (rem);
 
-       msg->reserved = htons(msg->reserved);
-       msg->len = htons(msg->len);
+       return NULL;
+}
 
-       for (; i != msg->num_opts; ++i) {
-               msg->opts[i].len = htons(msg->opts[i].len);
-               msg->opts[i].type = htons(msg->opts[i].type);
+static char *msg_filename(struct nmrp_msg *msg)
+{
+       static char buf[256];
+       uint16_t len;
+       char *p = msg_opt(msg, NMRP_O_FILE_NAME, &len);
+       if (p) {
+               len = MIN(sizeof(buf) - 1, len);
+               memcpy(buf, p, len);
+               buf[len] = '\0';
+               return buf;
        }
+
+       return NULL;
 }
 
-static void msg_hdr_ntoh(struct nmrp_msg *msg)
+static inline void msg_init(struct nmrp_msg *msg, uint16_t code)
 {
-       msg->reserved = ntohs(msg->reserved);
-       msg->len = ntohs(msg->len);
+       memset(msg, 0, sizeof(*msg));
+       msg->len = htons(NMRP_HDR_LEN);
+       msg->code = code;
 }
 
-static int msg_ntoh(struct nmrp_msg *msg)
+static char *msg_mkopt(struct nmrp_msg *msg, char *p, uint16_t type, const void *val, size_t len)
 {
-       struct nmrp_opt *opt = msg->opts;
-       int remaining;
+       struct nmrp_opt* opt = (struct nmrp_opt*)p;
 
-       remaining = msg->len - NMRP_HDR_LEN;
+       len &= 0xffff;
 
-       // FIXME maximum of two options supported, maximum option
-       // size is 12
-       if (remaining < NMRP_MAX_OPT_NUM * NMRP_MAX_OPT_SIZE) {
-               while (remaining > 0) {
-                       if (remaining < NMRP_OPT_LEN) {
-                               break;
-                       }
+       msg->len = ntohs(msg->len);
 
-                       opt->type = ntohs(opt->type);
-                       opt->len = ntohs(opt->len);
+       if ((msg->len + len > sizeof(*msg))) {
+               fprintf(stderr, "Error: invalid option - this is a bug\n");
+               exit(1);
+       }
 
-                       if (opt->len > NMRP_MAX_OPT_SIZE) {
-                               break;
-                       }
+       opt->type = htons(type);
+       opt->len = NMRP_OPT_HDR_LEN + len;
 
-                       remaining -= opt->len;
-                       ++opt;
-               }
+       if (val) {
+               memcpy(opt->val, val, len);
+       }
 
-               if (!remaining) {
-                       return 0;
-               }
+       msg->len += opt->len;
+       p += opt->len;
+
+       msg->len = htons(msg->len);
+       opt->len = htons(opt->len);
+
+       return p;
+}
+
+static void msg_mkadvertise(struct nmrp_msg *msg, const char *magic)
+{
+       msg_init(msg, NMRP_C_ADVERTISE);
+       msg_mkopt(msg, msg->opts, NMRP_O_MAGIC_NO, magic, strlen(magic));
+}
+
+static void msg_mkconfack(struct nmrp_msg *msg, uint32_t ipaddr, uint32_t ipmask, uint16_t region)
+{
+       char *p;
+       uint32_t ip[2] = { ipaddr, ipmask };
+
+       msg_init(msg, NMRP_C_CONF_ACK);
+       p = msg_mkopt(msg, msg->opts, NMRP_O_DEV_IP, &ip, 8);
+       p = msg_mkopt(msg, p, NMRP_O_FW_UP, NULL, 0);
+
+#ifdef NMRPFLASH_SET_REGION
+       if (region) {
+               p = msg_mkopt(msg, p, NMRP_O_DEV_REGION, &region, 2);
        }
+#endif
+}
 
-       fprintf(stderr, "Unexpected message format.\n");
-       msg_dump(msg, 0);
-       return 1;
+#ifdef NMRPFLASH_FUZZ
+#define NMRP_INITIAL_TIMEOUT 0
+#define ethsock_create(a, b) ((struct ethsock*)1)
+#define ethsock_get_hwaddr(a) ethsock_get_hwaddr_fake(a)
+#define ethsock_recv(sock, buf, len) read(STDIN_FILENO, buf, len)
+#define ethsock_send(a, b, c) (0)
+#define ethsock_set_timeout(a, b) (0)
+#define ethsock_arp_add(a, b, c, d) (0)
+#define ethsock_arp_del(a, b) (0)
+#define ethsock_ip_add(a, b, c, d) (0)
+#define ethsock_ip_del(a, b) (0)
+#define ethsock_close(a) (0)
+#define tftp_put(a) (0)
+
+static uint8_t *ethsock_get_hwaddr_fake(struct ethsock* sock)
+{
+       static uint8_t hwaddr[6] = { 0xfa, 0xfa, 0xfa, 0xfa, 0xfa, 0xfa };
+       return hwaddr;
 }
+#else
+#define NMRP_INITIAL_TIMEOUT 60
+#endif
 
 static int pkt_send(struct ethsock *sock, struct nmrp_pkt *pkt)
 {
-       size_t len = ntohs(pkt->msg.len) + sizeof(pkt->eh);
-       return ethsock_send(sock, pkt, len);
+       return ethsock_send(sock, pkt, sizeof(*pkt));
 }
 
 static int pkt_recv(struct ethsock *sock, struct nmrp_pkt *pkt)
 {
-       ssize_t bytes, len;
+       ssize_t bytes, mlen;
 
        memset(pkt, 0, sizeof(*pkt));
        bytes = ethsock_recv(sock, pkt, sizeof(*pkt));
@@ -205,21 +266,22 @@ static int pkt_recv(struct ethsock *sock, struct nmrp_pkt *pkt)
                return 1;
        } else if (!bytes) {
                return 2;
-       } else if (bytes < NMRP_MIN_PKT_LEN) {
-               fprintf(stderr, "Short packet (%d bytes)\n", (int)bytes);
-               return 1;
        }
 
-       msg_hdr_ntoh(&pkt->msg);
-       len = pkt->msg.len + sizeof(pkt->eh);
+       mlen = ntohs(pkt->msg.len);
 
-       if (bytes < len) {
-               fprintf(stderr, "Short packet (expected %d, got %d).\n",
-                               (int)len, (int)bytes);
+       if (bytes < (mlen + sizeof(pkt->eh))
+                       || bytes < NMRP_MIN_PKT_LEN
+                       || mlen < NMRP_HDR_LEN) {
+               fprintf(stderr, "Short packet (%d raw, %d message)\n",
+                               (int)bytes, (int)mlen);
                return 1;
+       } else if (mlen > sizeof(pkt->msg)) {
+               printf("Truncating %d byte message.\n", (int)mlen);
+               pkt->msg.len = htons(sizeof(pkt->msg));
        }
 
-       return msg_ntoh(&pkt->msg);
+       return 0;
 }
 
 static int mac_parse(const char *str, uint8_t *hwaddr)
@@ -246,16 +308,43 @@ static int mac_parse(const char *str, uint8_t *hwaddr)
        return 0;
 }
 
-static struct ethsock *gsock = NULL;
+struct is_valid_ip_arg
+{
+       struct in_addr *ipaddr;
+       struct in_addr *ipmask;
+       int result;
+};
 
-static void sigh(int sig)
+static int is_valid_ip_cb(struct ethsock_ip_callback_args *args)
 {
-       printf("\n");
-       if (gsock) {
-               ethsock_close(gsock);
+#define SUBNET(x) ((x)->ipaddr->s_addr & (x)->ipmask->s_addr)
+       struct is_valid_ip_arg *arg = args->arg;
+       if (SUBNET(args) == SUBNET(arg)) {
+               arg->result = args->ipaddr->s_addr != arg->ipaddr->s_addr;
+               return 0;
        }
 
-       exit(1);
+       return 1;
+#undef SUBNET
+}
+
+static int is_valid_ip(struct ethsock *sock, struct in_addr *ipaddr,
+               struct in_addr *ipmask)
+{
+       int status;
+       struct is_valid_ip_arg arg = {
+               .ipaddr = ipaddr,
+               .ipmask = ipmask,
+               .result = 0
+       };
+
+       status = ethsock_for_each_ip(sock, is_valid_ip_cb, &arg);
+       return status < 0 ? status : arg.result;
+}
+
+static void sigh(int sig)
+{
+       g_interrupted = 1;
 }
 
 static const char *spinner = "\\|/-";
@@ -264,11 +353,17 @@ int nmrp_do(struct nmrpd_args *args)
 {
        struct nmrp_pkt tx, rx;
        uint8_t *src, dest[6];
-       struct in_addr ipaddr, ipmask;
+       uint16_t region;
+       char *filename;
        time_t beg;
-       int i, err, ulreqs, expect;
+       int i, timeout, status, ulreqs, expect, upload_ok, autoip, kareqs;
        struct ethsock *sock;
+       struct ethsock_ip_undo *ip_undo = NULL;
+       struct ethsock_arp_undo *arp_undo = NULL;
+       uint32_t intf_addr;
        void (*sigh_orig)(int);
+       struct in_addr ipaddr;
+       struct in_addr ipmask;
 
        if (args->op != NMRP_UPLOAD_FW) {
                fprintf(stderr, "Operation not implemented.\n");
@@ -280,31 +375,95 @@ int nmrp_do(struct nmrpd_args *args)
                return 1;
        }
 
+       ipmask.s_addr = inet_addr(args->ipmask);
+       if (ipmask.s_addr == INADDR_NONE
+                       || netmask(bitcount(ipmask.s_addr)) != ipmask.s_addr) {
+               fprintf(stderr, "Invalid subnet mask '%s'.\n", args->ipmask);
+               return 1;
+       }
+
+       if (!args->ipaddr) {
+               autoip = true;
+               /* A random IP address. The MAC of the first device that was
+                * used to test this utility starts with a4:2b:8c, so we use
+                * 164 (0xa4) and 183 (0x2b + 0x8c).
+                *
+                * These addresses should not cause collisions on most networks,
+                * and if they do, the user is probably "poweruser" enough to
+                * be able to use the -a and -A options.
+                */
+               args->ipaddr = "10.164.183.252";
+
+               if (!args->ipaddr_intf) {
+                       args->ipaddr_intf = "10.164.183.253";
+               }
+       } else if (args->ipaddr_intf) {
+               autoip = true;
+       } else {
+               autoip = false;
+       }
+
        if ((ipaddr.s_addr = inet_addr(args->ipaddr)) == INADDR_NONE) {
                fprintf(stderr, "Invalid IP address '%s'.\n", args->ipaddr);
                return 1;
        }
 
-       if ((ipmask.s_addr = inet_addr(args->ipmask)) == INADDR_NONE) {
-               fprintf(stderr, "Invalid subnet mask '%s'.\n", args->ipmask);
+       if (args->ipaddr_intf && (intf_addr = inet_addr(args->ipaddr_intf)) == INADDR_NONE) {
+               fprintf(stderr, "Invalid IP address '%s'.\n", args->ipaddr_intf);
                return 1;
        }
 
-       if (strcmp(args->filename, "-") && access(args->filename, R_OK) == -1) {
-               fprintf(stderr, "Error accessing file '%s'.\n", args->filename);
+       if (args->file_local && strcmp(args->file_local, "-") && access(args->file_local, R_OK) == -1) {
+               fprintf(stderr, "Error accessing file '%s'.\n", args->file_local);
                return 1;
        }
 
-       err = 1;
+       if (args->file_remote) {
+               if (!tftp_is_valid_filename(args->file_remote)) {
+                       fprintf(stderr, "Invalid remote filename '%s'.\n",
+                                       args->file_remote);
+                       return 1;
+               }
+       }
+
+       if (args->region) {
+               region = to_region_code(args->region);
+               if (!region) {
+                       fprintf(stderr, "Invalid region code '%s'.\n", args->region);
+                       return 1;
+               }
+       } else {
+               region = 0;
+       }
+
+       status = 1;
 
        sock = ethsock_create(args->intf, ETH_P_NMRP);
        if (!sock) {
                return 1;
        }
 
-       gsock = sock;
        sigh_orig = signal(SIGINT, sigh);
 
+       if (!autoip) {
+               status = is_valid_ip(sock, &ipaddr, &ipmask);
+               if (status <= 0) {
+                       if (!status) {
+                               fprintf(stderr, "Address %s/%s cannot be used on interface %s.\n",
+                                               args->ipaddr, args->ipmask, args->intf);
+                       }
+                       goto out;
+               }
+       } else {
+               if (verbosity) {
+                       printf("Adding %s to interface %s.\n", args->ipaddr_intf, args->intf);
+               }
+
+               if (ethsock_ip_add(sock, intf_addr, ipmask.s_addr, &ip_undo) != 0) {
+                       goto out;
+               }
+       }
+
        if (ethsock_set_timeout(sock, args->rx_timeout)) {
                goto out;
        }
@@ -318,120 +477,173 @@ int nmrp_do(struct nmrpd_args *args)
        memcpy(tx.eh.ether_dhost, dest, 6);
        tx.eh.ether_type = htons(ETH_P_NMRP);
 
-       tx.msg.reserved = 0;
-       tx.msg.code = NMRP_C_ADVERTISE;
-       tx.msg.id = 0;
-       tx.msg.num_opts = 1;
-       tx.msg.opts[0].type = NMRP_O_MAGIC_NO;
-       tx.msg.opts[0].len = NMRP_OPT_LEN + 4;
-       tx.msg.opts[0].val.magic[0] = 'N';
-       tx.msg.opts[0].val.magic[1] = 'T';
-       tx.msg.opts[0].val.magic[2] = 'G';
-       tx.msg.opts[0].val.magic[3] = 'R';
-
-       msg_update_len(&tx.msg);
-       msg_hton(&tx.msg);
+       msg_mkadvertise(&tx.msg, "NTGR");
 
        i = 0;
-       beg = time(NULL);
+       upload_ok = 0;
+       timeout = args->blind ? 10 : NMRP_INITIAL_TIMEOUT;
+       beg = time_monotonic();
 
-       while (1) {
-               printf("\rAdvertising NMRP server on interface ... %c",
-                               spinner[i]);
+       while (!g_interrupted) {
+               printf("\rAdvertising NMRP server on %s ... %c",
+                               args->intf, spinner[i]);
                fflush(stdout);
                i = (i + 1) & 3;
 
                if (pkt_send(sock, &tx) < 0) {
-                       perror("sendto");
                        goto out;
                }
 
-               err = pkt_recv(sock, &rx);
-               if (err == 0 && memcmp(rx.eh.ether_dhost, src, 6) == 0) {
-                       break;
-               } else if (err == 1) {
-                       printf("ERR\n");
+               status = pkt_recv(sock, &rx);
+               if (status == 0) {
+                       if (memcmp(rx.eh.ether_dhost, src, 6) == 0) {
+                               break;
+                       } else if (verbosity) {
+                               printf("\nIgnoring bogus response: %s -> %s.\n",
+                                               mac_to_str(rx.eh.ether_shost),
+                                               mac_to_str(rx.eh.ether_dhost));
+                       }
+               } else if (status == 1) {
                        goto out;
                } else {
-                       if ((time(NULL) - beg) >= 60) {
-                               printf("\nNo response after 60 seconds. Bailing out.\n");
-                               goto out;
+                       /* because we don't want nmrpflash's exit status to be zero */
+                       status = 1;
+                       if ((time_monotonic() - beg) >= timeout) {
+                               printf("\nNo response after %d seconds. ", timeout);
+                               if (!args->blind) {
+                                       printf("Bailing out.\n");
+                                       goto out;
+                               } else {
+                                       // we're blind, so fake a response from the MAC specified by -m
+                                       memcpy(rx.eh.ether_shost, dest, 6);
+                                       msg_init(&rx.msg, NMRP_C_CONF_REQ);
+                                       printf("Faking one.");
+                                       break;
+                               }
                        }
                }
        }
 
        printf("\n");
 
+       memcpy(tx.eh.ether_dhost, rx.eh.ether_shost, 6);
+
+       if (ethsock_arp_add(sock, rx.eh.ether_shost, ipaddr.s_addr, &arp_undo) != 0) {
+               goto out;
+       }
+
        expect = NMRP_C_CONF_REQ;
        ulreqs = 0;
+       kareqs = 0;
 
-       do {
+       while (!g_interrupted) {
                if (expect != NMRP_C_NONE && rx.msg.code != expect) {
-                       fprintf(stderr, "Received code 0x%02x while waiting for 0x%02x!\n",
-                                       rx.msg.code, expect);
+                       fprintf(stderr, "Received %s while waiting for %s!\n",
+                                       msg_code_str(rx.msg.code), msg_code_str(expect));
                }
 
-               tx.msg.code = NMRP_C_NONE;
-               tx.msg.reserved = 0;
-               tx.msg.id = 0;
-               tx.msg.num_opts = 0;
-               tx.msg.len = 0;
+               msg_init(&tx.msg, NMRP_C_NONE);
 
-               err = 1;
+               status = 1;
 
                switch (rx.msg.code) {
                        case NMRP_C_ADVERTISE:
                                printf("Received NMRP advertisement from %s.\n",
                                                mac_to_str(rx.eh.ether_shost));
-                               err = 1;
+                               status = 1;
                                goto out;
                        case NMRP_C_CONF_REQ:
-                               tx.msg.code = NMRP_C_CONF_ACK;
-                               tx.msg.num_opts = 2;
-
-                               tx.msg.opts[0].type = NMRP_O_DEV_IP;
-                               tx.msg.opts[0].len = NMRP_OPT_LEN + 2 * 4;
-
-                               memcpy(tx.msg.opts[0].val.ip.addr, &ipaddr, 4);
-                               memcpy(tx.msg.opts[0].val.ip.mask, &ipmask, 4);
-
-                               tx.msg.opts[1].type = NMRP_O_FW_UP;
-                               tx.msg.opts[1].len = NMRP_OPT_LEN;
-
+                               msg_mkconfack(&tx.msg, ipaddr.s_addr, ipmask.s_addr, region);
                                expect = NMRP_C_TFTP_UL_REQ;
 
                                printf("Received configuration request from %s.\n",
                                                mac_to_str(rx.eh.ether_shost));
 
-                               memcpy(tx.eh.ether_dhost, rx.eh.ether_shost, 6);
-
-                               printf("Sending configuration: ip %s, mask %s.\n",
-                                               args->ipaddr, args->ipmask);
+                               printf("Sending configuration: %s/%d.\n",
+                                               args->ipaddr, bitcount(ipmask.s_addr));
 
                                break;
                        case NMRP_C_TFTP_UL_REQ:
-                               if (++ulreqs > 5) {
-                                       fprintf(stderr, "Device re-requested file upload %d "
-                                                       "times; aborting.\n", ulreqs);
-                                       tx.msg.code = NMRP_C_CLOSE_REQ;
+                               if (!upload_ok) {
+                                       if (++ulreqs > 5) {
+                                               printf("Bailing out after %d upload requests.\n",
+                                                               ulreqs);
+                                               tx.msg.code = NMRP_C_CLOSE_REQ;
+                                               break;
+                                       }
+                               } else {
+                                       if (verbosity) {
+                                               printf("Ignoring extra upload request.\n");
+                                       }
+                                       ethsock_set_timeout(sock, args->ul_timeout);
+                                       tx.msg.code = NMRP_C_KEEP_ALIVE_REQ;
                                        break;
                                }
 
-                               if (!args->tftpcmd) {
-                                       printf("Uploading %s ... ", args->filename);
-                                       fflush(stdout);
-                                       err = tftp_put(args);
-                               } else {
-                                       printf("Running %s ... ", args->tftpcmd);
+                               filename = msg_filename(&rx.msg);
+                               if (filename) {
+                                       if (!args->file_remote) {
+                                               args->file_remote = filename;
+                                       }
+                                       printf("Received upload request: filename '%s'.\n", filename);
+                               } else if (!args->file_remote) {
+                                       args->file_remote = leafname(args->file_local);
+                                       printf("Received upload request without filename.\n");
+                               }
+
+                               status = 0;
+
+                               if (args->tftpcmd) {
+                                       printf("Executing '%s' ... \n", args->tftpcmd);
+                                       setenv("IP", inet_ntoa(ipaddr), 1);
+                                       setenv("PORT", lltostr(args->port, 10), 1);
+                                       setenv("MAC", mac_to_str(rx.eh.ether_shost), 1);
+                                       setenv("NETMASK", inet_ntoa(ipmask), 1);
+                                       //setenv("FILENAME", args->file_remote ? args->file_remote : "", 1);
+                                       status = system(args->tftpcmd);
+                               }
+
+                               if (!status && args->file_local) {
+                                       if (!autoip) {
+                                               status = is_valid_ip(sock, &ipaddr, &ipmask);
+                                               if (status < 0) {
+                                                       goto out;
+                                               } else if (!status) {
+                                                       printf("IP address of %s has changed. Please assign a "
+                                                                       "static ip to the interface.\n", args->intf);
+                                                       tx.msg.code = NMRP_C_CLOSE_REQ;
+                                                       break;
+                                               }
+                                       }
+
+                                       if (verbosity) {
+                                               printf("Using remote filename '%s'.\n",
+                                                               args->file_remote);
+                                       }
+
+                                       if (!strcmp(args->file_local, "-")) {
+                                               printf("Uploading from stdin ... ");
+                                       } else {
+                                               printf("Uploading %s ... ", leafname(args->file_local));
+                                       }
                                        fflush(stdout);
-                                       err = system(args->tftpcmd);
+                                       if (!(status = tftp_put(args))) {
+                                               printf("OK\n");
+                                       }
+
                                }
 
-                               if (!err) {
-                                       printf("OK\nWaiting for remote to respond.\n");
+                               if (!status) {
+                                       if (args->blind) {
+                                               goto out;
+                                       }
+
+                                       printf("Waiting for remote to respond.\n");
+                                       upload_ok = 1;
                                        ethsock_set_timeout(sock, args->ul_timeout);
-                                       expect = NMRP_C_CLOSE_REQ;
-                               } else if (err == -2) {
+                                       tx.msg.code = NMRP_C_KEEP_ALIVE_REQ;
+                                       expect = NMRP_C_NONE;
+                               } else if (status == -2) {
                                        expect = NMRP_C_TFTP_UL_REQ;
                                } else {
                                        goto out;
@@ -440,51 +652,74 @@ int nmrp_do(struct nmrpd_args *args)
                                break;
                        case NMRP_C_KEEP_ALIVE_REQ:
                                tx.msg.code = NMRP_C_KEEP_ALIVE_ACK;
+                               ethsock_set_timeout(sock, args->ul_timeout);
+                               printf("\rReceived keep-alive request (%d).  ", ++kareqs);
                                break;
                        case NMRP_C_CLOSE_REQ:
                                tx.msg.code = NMRP_C_CLOSE_ACK;
                                break;
                        case NMRP_C_CLOSE_ACK:
-                               err = 0;
+                               status = 0;
                                goto out;
                        default:
                                fprintf(stderr, "Unknown message code 0x%02x!\n",
                                                rx.msg.code);
-                               msg_dump(&rx.msg, 0);
+                               msg_dump(&rx.msg);
                }
 
                if (tx.msg.code != NMRP_C_NONE) {
-                       msg_update_len(&tx.msg);
-                       msg_hton(&tx.msg);
-
                        if (pkt_send(sock, &tx) < 0) {
-                               perror("sendto");
+                               goto out;
+                       }
+
+                       if (tx.msg.code == NMRP_C_CLOSE_REQ) {
                                goto out;
                        }
                }
 
                if (rx.msg.code == NMRP_C_CLOSE_REQ) {
+                       if (kareqs) {
+                               printf("\n");
+                       }
+
                        printf("Remote finished. Closing connection.\n");
                        break;
                }
 
-               err = pkt_recv(sock, &rx);
-               if (err) {
-                       if (err == 2) {
-                               fprintf(stderr, "Timeout while waiting for 0x%02x.\n", expect);
+               status = pkt_recv(sock, &rx);
+               if (status) {
+                       if (status == 2) {
+                               fprintf(stderr, "Timeout while waiting for %s. ",
+                                               msg_code_str(expect));
+                       }
+
+                       if (!args->blind) {
+                               printf("\n");
+                               goto out;
+                       } else {
+                               printf("Faking response.\n");
+                               msg_init(&rx.msg, expect);
+                               memcpy(rx.eh.ether_shost, tx.eh.ether_dhost, 6);
                        }
-                       goto out;
                }
 
                ethsock_set_timeout(sock, args->rx_timeout);
 
-       } while (1);
+       }
 
-       err = 0;
+       if (!g_interrupted) {
+               status = 0;
+               if (ulreqs) {
+                       printf("Reboot your device now.\n");
+               } else {
+                       printf("No upload request received.\n");
+               }
+       }
 
 out:
        signal(SIGINT, sigh_orig);
-       gsock = NULL;
+       ethsock_arp_del(sock, &arp_undo);
+       ethsock_ip_del(sock, &ip_undo);
        ethsock_close(sock);
-       return err;
+       return status;
 }