On Tue, Oct 04, 2011 at 10:53:20PM -0700, Alan Coopersmith wrote:
> commit a18460b385ae03 converted from a fixed maximum number of devices
> to dynamically growing the list via realloc, but didn't update the
> error message shown on failure.
> 
> Signed-off-by: Alan Coopersmith <[email protected]>
> ---
>  src/solx_devfs.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/solx_devfs.c b/src/solx_devfs.c
> index d47a846..ea91479 100644
> --- a/src/solx_devfs.c
> +++ b/src/solx_devfs.c
> @@ -454,8 +454,9 @@ probe_dev(nexus_t *nexus, pcitool_reg_t *prg_p, 
> probe_info_t *pinfo)
>                       new_num_elems * sizeof (struct pci_device_private));
>               if (new_devs == NULL) {
>                   (void) fprintf(stderr,
> -                                "Maximum number of PCI devices found,"
> -                                " discarding additional devices\n");
> +                                "Error allocating memory for PCI devices:"
> +                                " %s\n discarding additional devices\n",
> +                                strerror(errno));
>                   return (rval);
>               }
>               (void) memset(&new_devs[pinfo->num_devices], 0,
> -- 
> 1.7.3.2
> 

Reviewed-by: Dirk Wallenstein <[email protected]>

Is the initial space and lower case in the second line intentional?

-- 
Cheers,
  Dirk
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to