change type of result variables to ssize_t

[libserialport.git] / freebsd.c

diff --git a/freebsd.c b/freebsd.c
index f7957edfe8ffe385b1d2b937fd4ca6d02aaae29c..b51b731b6071fc3523e0e2369d861c8822130a48 100644 (file)
--- a/freebsd.c
+++ b/freebsd.c
@@ -76,6 +76,7 @@
  *    device description.
  */
 
+#include <config.h>
 #include <unistd.h>
 #include <stdint.h>
 #include <stdlib.h>
@@ -133,7 +134,7 @@ static int libusb_query_port(struct libusb20_device *dev, int idx,
        if (sbuf[0] == 0)
                return rc;
 
-       DBG("device interface descriptor: idx=%003d '%s'\n", idx, sbuf);
+       DBG("Device interface descriptor: idx=%003d '%s'\n", idx, sbuf);
        j = strchr(sbuf, ':');
        if (j > sbuf) {
                sbuf[j - sbuf] = 0;
@@ -244,13 +245,13 @@ SP_PRIV enum sp_return get_port_details(struct sp_port *port)
        uint8_t idx;
        int sub_inst;
 
-       DBG("portname: '%s'\n", port->name);
+       DBG("Portname: '%s'\n", port->name);
 
        if (!strncmp(port->name, DEV_CUA_PATH, strlen(DEV_CUA_PATH))) {
                cua_sfx = port->name + strlen(DEV_CUA_PATH);
                DBG("'%s': '%s'\n", DEV_CUA_PATH, cua_sfx);
        } else {
-               RETURN_ERROR(SP_ERR_ARG, "Device name not recognized.");
+               RETURN_ERROR(SP_ERR_ARG, "Device name not recognized");
        }
 
        /* Native UART enumeration. */
@@ -271,7 +272,7 @@ SP_PRIV enum sp_return get_port_details(struct sp_port *port)
                        break;
 
                libusb20_dev_open(dev, 0);
-               DBG("device descriptor: '%s'\n", libusb20_dev_get_desc(dev));
+               DBG("Device descriptor: '%s'\n", libusb20_dev_get_desc(dev));
 
                for (idx = 0; idx <= UINT8_MAX - 1; idx++) {
                        char *drv_name_str = NULL;
@@ -316,7 +317,7 @@ SP_PRIV enum sp_return get_port_details(struct sp_port *port)
        libusb20_be_free(be);
 
        if (cua_dev_found == 0)
-               DBG("WARN: found no match '%s' %s'\n", port->name, cua_sfx);
+               DBG("WARN: Found no match '%s' %s'\n", port->name, cua_sfx);
 
        RETURN_OK();
 }
@@ -324,36 +325,35 @@ SP_PRIV enum sp_return get_port_details(struct sp_port *port)
 SP_PRIV enum sp_return list_ports(struct sp_port ***list)
 {
        DIR *dir;
-       struct dirent entry;
-       struct dirent *result;
+       struct dirent *entry;
        struct termios tios;
        char name[PATH_MAX];
        int fd, ret;
 
        DEBUG("Enumerating tty devices");
        if (!(dir = opendir("/dev")))
-               RETURN_FAIL("could not open dir /dev");
+               RETURN_FAIL("Could not open dir /dev");
 
        DEBUG("Iterating over results");
-       while (!readdir_r(dir, &entry, &result) && result) {
+       while ((entry = readdir(dir))) {
                ret = SP_OK;
-               if (entry.d_type != DT_CHR)
+               if (entry->d_type != DT_CHR)
                        continue;
-               if (strncmp(entry.d_name, "cuaU", 4) != 0)
-                       if (strncmp(entry.d_name, "cuau", 4) != 0)
-                               if (strncmp(entry.d_name, "cuad", 4) != 0)
+               if (strncmp(entry->d_name, "cuaU", 4) != 0)
+                       if (strncmp(entry->d_name, "cuau", 4) != 0)
+                               if (strncmp(entry->d_name, "cuad", 4) != 0)
                                        continue;
-               if (strend(entry.d_name, ".init"))
+               if (strend(entry->d_name, ".init"))
                        continue;
-               if (strend(entry.d_name, ".lock"))
+               if (strend(entry->d_name, ".lock"))
                        continue;
 
-               snprintf(name, sizeof(name), "/dev/%s", entry.d_name);
+               snprintf(name, sizeof(name), "/dev/%s", entry->d_name);
                DEBUG_FMT("Found device %s", name);
 
                /* Check that we can open tty/cua device in rw mode - we need that. */
                if ((fd = open(name, O_RDWR | O_NONBLOCK | O_NOCTTY | O_TTY_INIT | O_CLOEXEC)) < 0) {
-                       DEBUG("open failed, skipping");
+                       DEBUG("Open failed, skipping");
                        continue;
                }
 
@@ -369,11 +369,11 @@ SP_PRIV enum sp_return list_ports(struct sp_port ***list)
                        continue;
 
                DEBUG_FMT("Found port %s", name);
-               DBG("%s: %s\n", __func__, entry.d_name);
+               DBG("%s: %s\n", __func__, entry->d_name);
 
                *list = list_append(*list, name);
-               if (!list) {
-                       SET_ERROR(ret, SP_ERR_MEM, "list append failed");
+               if (!*list) {
+                       SET_ERROR(ret, SP_ERR_MEM, "List append failed");
                        break;
                }
        }