invalid error check in modbus_init_listen_tcp

[kylesch]

Hello,

I think that that the error checking on the return value of the accept() call in modbus_init_listen_tcp is checking the wrong variable. Here is a one-line patch that should fix the issue:

diff -Naur libmodbus-2.0.2.old/modbus/modbus.c libmodbus-2.0.2.new/modbus/modbus.c
--- libmodbus-2.0.2.old/modbus/modbus.c 2008-08-07 11:17:55.000000000 -0400
+++ libmodbus-2.0.2.new/modbus/modbus.c 2008-09-25 14:14:09.000000000 -0400
@@ -1827,7 +1827,7 @@

         addrlen = sizeof(struct sockaddr_in);
         mb_param->fd = accept(new_socket, (struct sockaddr *)&addr, &addrlen);
- if (ret < 0) {
+ if (mb_param->fd < 0) {
                 perror("accept");
                 close(new_socket);
                 new_socket = 0;

[Stéphane Raimbault (sra)]

Thank you for your bug report kylesch.