server: fix bug in going from thread_number to thread
authorJens Axboe <axboe@kernel.dk>
Wed, 28 Mar 2012 07:15:05 +0000 (09:15 +0200)
committerJens Axboe <axboe@kernel.dk>
Wed, 28 Mar 2012 07:15:05 +0000 (09:15 +0200)
thread_numbers start at 1, not 0.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
server.c

index 5f005500e7cb35fc3853ea259d4840be6c787d5f..8b91d26fe328c6577af7a4eace788dd8cb2bc16d 100644 (file)
--- a/server.c
+++ b/server.c
@@ -698,12 +698,12 @@ static int handle_update_job_cmd(struct fio_net_cmd *cmd)
 
        dprint(FD_NET, "server: updating options for job %u\n", tnumber);
 
-       if (tnumber >= thread_number) {
+       if (!tnumber || tnumber > thread_number) {
                send_update_job_reply(server_fd, cmd->tag, ENODEV);
                return 0;
        }
 
-       td = &threads[tnumber];
+       td = &threads[tnumber - 1];
        convert_thread_options_to_cpu(&td->o, &pdu->top);
        send_update_job_reply(server_fd, cmd->tag, 0);
        return 0;