Date: Tue, 16 Jun 2009 17:45:32 GMT From: Sylvestre Gallon <syl@FreeBSD.org> To: Perforce Change Reviews <perforce@FreeBSD.org> Subject: PERFORCE change 164519 for review Message-ID: <200906161745.n5GHjWbY076344@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=164519 Change 164519 by syl@syl_atuin on 2009/06/16 17:45:06 Fix debug. Remove unintialised variable and possible bad derefence. Now all the test rework. Affected files ... .. //depot/projects/soc2009/syl_usb/src/lib/libusb/libusb10.c#36 edit .. //depot/projects/soc2009/syl_usb/src/lib/libusb/libusb10_desc.c#16 edit Differences ... ==== //depot/projects/soc2009/syl_usb/src/lib/libusb/libusb10.c#36 (text+ko) ==== @@ -160,24 +160,29 @@ if (usb_backend == NULL) return (-1); + printf("1\n"); pdev = NULL; i = 0; while ((pdev = libusb20_be_device_foreach(usb_backend, pdev))) i++; + printf("2\n"); if (list == NULL) { libusb20_be_free(usb_backend); return (LIBUSB_ERROR_INVALID_PARAM); } *list = malloc((i + 1) * sizeof(void *)); + printf("3\n"); if (*list == NULL) { libusb20_be_free(usb_backend); return (LIBUSB_ERROR_NO_MEM); } + printf("4\n"); i = 0; while ((pdev = libusb20_be_device_foreach(usb_backend, NULL))) { /* get device into libUSB v1.0 list */ libusb20_be_dequeue_device(usb_backend, pdev); + printf("5\n"); ddesc = libusb20_dev_get_device_desc(pdev); dev = malloc(sizeof(*dev)); @@ -205,6 +210,7 @@ i++; } (*list)[i] = NULL; + printf("6\n"); libusb20_be_free(usb_backend); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_device_list leave"); @@ -221,6 +227,7 @@ int i; libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_free_device_list enter"); @@ -240,6 +247,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_bus_number enter"); @@ -254,6 +262,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_device_address enter"); @@ -273,6 +282,7 @@ libusb_context *ctx; int i, j, k, ret; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_max_packet_size enter"); @@ -308,6 +318,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_ref_device enter"); @@ -327,6 +338,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_unref_device enter"); @@ -510,6 +522,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_device enter"); @@ -525,6 +538,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_configuration enter"); @@ -548,6 +562,7 @@ struct libusb20_device *pdev; libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_set_configuration enter"); @@ -568,6 +583,7 @@ libusb_context *ctx; int ret = 0; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_claim_interface enter"); @@ -595,6 +611,7 @@ libusb_context *ctx; int ret; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_release_interface enter"); @@ -624,6 +641,7 @@ libusb_context *ctx; int ret; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_set_interface_alt_setting enter"); @@ -655,6 +673,7 @@ libusb_context *ctx; int ret; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_clear_halt enter"); @@ -681,6 +700,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_reset_device enter"); @@ -697,6 +717,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_kernel_driver_active enter"); @@ -713,6 +734,7 @@ struct libusb20_device *pdev; libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_detach_kernel_driver enter"); @@ -736,6 +758,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_attach_kernel_driver enter"); @@ -756,6 +779,7 @@ libusb_context *ctx; int len; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_alloc_transfer enter"); @@ -783,6 +807,7 @@ struct usb_transfer *bxfer; libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_free_transfer enter"); @@ -1086,6 +1111,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_cancel_transfer enter"); ==== //depot/projects/soc2009/syl_usb/src/lib/libusb/libusb10_desc.c#16 (text+ko) ==== @@ -44,6 +44,7 @@ struct libusb20_device *pdev; libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_device_descriptor enter"); @@ -80,6 +81,7 @@ libusb_context *ctx; uint8_t idx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_active_config_descriptor enter"); @@ -108,6 +110,7 @@ uint8_t nif, nend, nalt, i, j, k; uint32_t if_idx, endp_idx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_config_descriptor enter"); @@ -235,8 +238,10 @@ libusb_context *ctx; int i; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_config_descriptor_by_value enter"); + if (dev == NULL || config == NULL) return (LIBUSB_ERROR_INVALID_PARAM); @@ -262,6 +267,7 @@ { libusb_context *ctx; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_free_config_descriptor enter"); @@ -277,6 +283,7 @@ libusb_context *ctx; int ret; + ctx = NULL; GET_CONTEXT(ctx); dprintf(ctx, LIBUSB_DEBUG_FUNCTION, "libusb_get_string_descriptor_ascii enter");
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200906161745.n5GHjWbY076344>