Changes in uspace/lib/usbdev/src/altiface.c [ab27e01:e978ada] in mainline

File:

• uspace/lib/usbdev/src/altiface.c (modified) (5 diffs)

uspace/lib/usbdev/src/altiface.c

@@ -90 +90 @@
  * @return Error code.
  */
-int usb_alternate_interfaces_init(usb_alternate_interfaces_t *alternates,
-    const uint8_t *config_descr, size_t config_descr_size, int interface_number)
+int usb_alternate_interfaces_create(const uint8_t *config_descr,
+    size_t config_descr_size, int interface_number,
+    usb_alternate_interfaces_t **alternates_ptr)
 {
-        assert(alternates != NULL);
+        assert(alternates_ptr != NULL);
         assert(config_descr != NULL);
         assert(config_descr_size > 0);
 
-        alternates->alternatives = NULL;
-        alternates->alternative_count = 0;
-        alternates->current = 0;
-
+        *alternates_ptr = NULL;
         if (interface_number < 0) {
                 return EOK;
+        }
+
+        usb_alternate_interfaces_t *alternates
+            = malloc(sizeof(usb_alternate_interfaces_t));
+        if (alternates == NULL) {
+                return ENOMEM;
         }
 
@@ -110 +114 @@
 
         if (alternates->alternative_count == 0) {
+                free(alternates);
                 return ENOENT;
         }
@@ -116 +121 @@
             sizeof(usb_alternate_interface_descriptors_t));
         if (alternates->alternatives == NULL) {
+                free(alternates);
                 return ENOMEM;
         }
 
-        const usb_dp_parser_t dp_parser = {
+        alternates->current = 0;
+
+        usb_dp_parser_t dp_parser = {
                 .nesting = usb_dp_standard_descriptor_nesting
         };
-        const usb_dp_parser_data_t dp_data = {
+        usb_dp_parser_data_t dp_data = {
                 .data = config_descr,
                 .size = config_descr_size,
@@ -139 +147 @@
                     || (iface->interface_number != interface_number)) {
                         iface_ptr = usb_dp_get_sibling_descriptor(&dp_parser,
-                            &dp_data, dp_data.data, iface_ptr);
+                            &dp_data,
+                            dp_data.data, iface_ptr);
                         continue;
                 }
@@ -161 +170 @@
         }
 
+        *alternates_ptr = alternates;
+
         return EOK;
 }
 
-void usb_alternate_interfaces_deinit(usb_alternate_interfaces_t *alternate)
+void usb_alternate_interfaces_destroy(usb_alternate_interfaces_t *alternate)
 {
         if (!alternate)
                 return;
         free(alternate->alternatives);
+        free(alternate);
 }
 /**