]> sigrok.org Git - libserialport.git/blobdiff - serialport.c
Port name string length now no longer needs to be passed around.
[libserialport.git] / serialport.c
index 6fc52688c6d4871bacf07cf3361f4843eae9d3b6..3c81f9ef097af937f250f2a5ef7a23789493327f 100644 (file)
 #include <sys/ioctl.h>
 #endif
 #ifdef __APPLE__
-#include <IOKitLib.h>
-#include <serial/IOSerialKeys.h>
+#include <IOKit/IOKitLib.h>
+#include <IOKit/serial/IOSerialKeys.h>
+#include <sys/syslimits.h>
 #endif
 #ifdef __linux__
 #include "libudev.h"
+#include "linux/serial.h"
 #endif
 
 #include "serialport.h"
 
-static char **sp_list_new(void)
+static struct sp_port *sp_port_new(const char *portname)
 {
-       char **list;
-       if ((list = malloc(sizeof(char *))))
-               list[0] = NULL;
-       return list;
+       struct sp_port *port;
+       int len;
+
+       if (!(port = malloc(sizeof(struct sp_port))))
+               return NULL;
+
+       len = strlen(portname) + 1;
+
+       if (!(port->name = malloc(len)))
+       {
+               free(port);
+               return NULL;
+       }
+
+       memcpy(port->name, portname, len);
+
+       return port;
 }
 
-static char **sp_list_append(char **list, void *data, size_t len)
+static struct sp_port **sp_list_append(struct sp_port **list, const char *portname)
 {
        void *tmp;
        unsigned int count;
        for (count = 0; list[count]; count++);
-       if (!(tmp = realloc(list, sizeof(char *) * (count + 2))))
+       if (!(tmp = realloc(list, sizeof(struct sp_port *) * (count + 2))))
                goto fail;
        list = tmp;
-       if (!(list[count] = malloc(len)))
+       if (!(list[count] = sp_port_new(portname)))
                goto fail;
-       memcpy(list[count], data, len);
        list[count + 1] = NULL;
        return list;
 fail:
@@ -74,48 +88,64 @@ fail:
  *
  * @return A null-terminated array of port name strings.
  */
-char **sp_list_ports(void)
+struct sp_port **sp_list_ports(void)
 {
-       char **list = NULL;
+       struct sp_port **list;
+
+       if (!(list = malloc(sizeof(struct sp_port **))))
+               return NULL;
+
+       list[0] = NULL;
 
 #ifdef _WIN32
        HKEY key;
-       TCHAR *name, *data;
-       DWORD max_name_len, max_data_size, max_data_len;
-       DWORD name_len, data_size, data_len;
+       TCHAR *value, *data;
+       DWORD max_value_len, max_data_size, max_data_len;
+       DWORD value_len, data_size, data_len;
        DWORD type, index = 0;
+       char *name;
+       int name_len;
 
        if (RegOpenKeyEx(HKEY_LOCAL_MACHINE, _T("HARDWARE\\DEVICEMAP\\SERIALCOMM"),
                        0, KEY_QUERY_VALUE, &key) != ERROR_SUCCESS)
                return NULL;
        if (RegQueryInfoKey(key, NULL, NULL, NULL, NULL, NULL, NULL, NULL,
-                               &max_name_len, &max_data_size, NULL, NULL) != ERROR_SUCCESS)
+                               &max_value_len, &max_data_size, NULL, NULL) != ERROR_SUCCESS)
                goto out_close;
        max_data_len = max_data_size / sizeof(TCHAR);
-       if (!(name = malloc((max_name_len + 1) * sizeof(TCHAR))))
+       if (!(value = malloc((max_value_len + 1) * sizeof(TCHAR))))
                goto out_close;
        if (!(data = malloc((max_data_len + 1) * sizeof(TCHAR))))
-               goto out_free_name;
-       if (!(list = sp_list_new()))
-               goto out;
+               goto out_free_value;
        while (
-               name_len = max_name_len,
+               value_len = max_value_len,
                data_size = max_data_size,
-               RegEnumValue(key, index, name, &name_len,
+               RegEnumValue(key, index, value, &value_len,
                        NULL, &type, (LPBYTE)data, &data_size) == ERROR_SUCCESS)
        {
                data_len = data_size / sizeof(TCHAR);
                data[data_len] = '\0';
+#ifdef UNICODE
+               name_len = WideCharToMultiByte(CP_ACP, 0, data, -1, NULL, 0, NULL, NULL)
+#else
+               name_len = data_len + 1;
+#endif
+               if (!(name = malloc(name_len)))
+                       goto out;
+#ifdef UNICODE
+               WideCharToMultiByte(CP_ACP, 0, data, -1, name, name_len, NULL, NULL);
+#else
+               strcpy(name, data);
+#endif
                if (type == REG_SZ)
-                       if (!(list = sp_list_append(list,
-                                       data, (data_len + 1) * sizeof(TCHAR))))
+                       if (!(list = sp_list_append(list, name)))
                                goto out;
                index++;
        }
 out:
        free(data);
-out_free_name:
-       free(name);
+out_free_value:
+       free(value);
 out_close:
        RegCloseKey(key);
        return list;
@@ -144,10 +174,7 @@ out_close:
        if (!(path = malloc(PATH_MAX)))
                goto out_release;
 
-       if (!(list = sp_list_new()))
-               goto out;
-
-       while (port = IOIteratorNext(iter)) {
+       while ((port = IOIteratorNext(iter))) {
                cf_path = IORegistryEntryCreateCFProperty(port,
                                CFSTR(kIOCalloutDeviceKey), kCFAllocatorDefault, 0);
                if (cf_path) {
@@ -155,7 +182,7 @@ out_close:
                                        path, PATH_MAX, kCFStringEncodingASCII);
                        CFRelease(cf_path);
                        if (result)
-                               if (!(list = sp_list_append(list, path, strlen(path) + 1)))
+                               if (!(list = sp_list_append(list, path)))
                                {
                                        IOObjectRelease(port);
                                        goto out;
@@ -176,22 +203,46 @@ out_release:
        struct udev_list_entry *ud_list;
        struct udev_list_entry *ud_entry;
        const char *path;
-       struct udev_device *ud_dev;
+       struct udev_device *ud_dev, *ud_parent;
        const char *name;
+       const char *driver;
+       int fd, ioctl_result;
+       struct serial_struct serial_info;
 
        ud = udev_new();
        ud_enumerate = udev_enumerate_new(ud);
        udev_enumerate_add_match_subsystem(ud_enumerate, "tty");
        udev_enumerate_scan_devices(ud_enumerate);
        ud_list = udev_enumerate_get_list_entry(ud_enumerate);
-       if (!(list = sp_list_new()))
-               goto out;
        udev_list_entry_foreach(ud_entry, ud_list)
        {
                path = udev_list_entry_get_name(ud_entry);
                ud_dev = udev_device_new_from_syspath(ud, path);
+               /* If there is no parent device, this is a virtual tty. */
+               ud_parent = udev_device_get_parent(ud_dev);
+               if (ud_parent == NULL)
+               {
+                       udev_device_unref(ud_dev);
+                       continue;
+               }
                name = udev_device_get_devnode(ud_dev);
-               list = sp_list_append(list, (void *)name, strlen(name) + 1);
+               /* The serial8250 driver has a hardcoded number of ports.
+                * The only way to tell which actually exist on a given system
+                * is to try to open them and make an ioctl call. */
+               driver = udev_device_get_driver(ud_parent);
+               if (driver && !strcmp(driver, "serial8250"))
+               {
+                       if ((fd = open(name, O_RDWR | O_NONBLOCK | O_NOCTTY)) < 0)
+                               goto skip;
+                       ioctl_result = ioctl(fd, TIOCGSERIAL, &serial_info);
+                       close(fd);
+                       if (ioctl_result != 0)
+                               goto skip;
+                       if (serial_info.type == PORT_UNKNOWN)
+                               goto skip;
+               }
+               list = sp_list_append(list, name);
+skip:
                udev_device_unref(ud_dev);
                if (!list)
                        goto out;
@@ -206,7 +257,7 @@ out:
 /**
  * Free a port list returned by sp_list_ports.
  */
-void sp_free_port_list(char **list)
+void sp_free_port_list(struct sp_port **list)
 {
        unsigned int i;
        for (i = 0; list[i]; i++)
@@ -241,16 +292,11 @@ static int sp_validate_port(struct sp_port *port)
  * @return SP_OK on success, SP_ERR_FAIL on failure,
  *         or SP_ERR_ARG if an invalid port or name is passed.
  */
-int sp_open(struct sp_port *port, char *portname, int flags)
+int sp_open(struct sp_port *port, int flags)
 {
        if (!port)
                return SP_ERR_ARG;
 
-       if (!portname)
-               return SP_ERR_ARG;
-
-       port->name = portname;
-
 #ifdef _WIN32
        DWORD desired_access = 0, flags_and_attributes = 0;
        /* Map 'flags' to the OS-specific settings. */
@@ -618,7 +664,7 @@ int sp_set_params(struct sp_port *port, int baudrate,
                return SP_ERR_ARG;
        }
 
-       term.c_iflag &= ~(IXON | IXOFF);
+       term.c_iflag &= ~(IXON | IXOFF | IXANY);
        term.c_cflag &= ~CRTSCTS;
        switch (flowcontrol) {
        case 0:
@@ -628,14 +674,14 @@ int sp_set_params(struct sp_port *port, int baudrate,
                term.c_cflag |= CRTSCTS;
                break;
        case 2:
-               term.c_iflag |= IXON | IXOFF;
+               term.c_iflag |= IXON | IXOFF | IXANY;
                break;
        default:
                return SP_ERR_ARG;
        }
 
        term.c_iflag &= ~IGNPAR;
-       term.c_cflag &= ~(PARODD | PARENB);
+       term.c_cflag &= ~(PARENB | PARODD);
        switch (parity) {
        case SP_PARITY_NONE:
                term.c_iflag |= IGNPAR;
@@ -660,6 +706,9 @@ int sp_set_params(struct sp_port *port, int baudrate,
        /* Disable canonical mode, and don't echo input characters. */
        term.c_lflag &= ~(ICANON | ECHO);
 
+       /* Ignore modem status lines; enable receiver */
+       term.c_cflag |= (CLOCAL | CREAD);
+
        /* Write the configured settings. */
        if (tcsetattr(port->fd, TCSADRAIN, &term) < 0)
                return SP_ERR_FAIL;