[PATCH 03/12] xf86: don't assign unused pointer.

Alan Coopersmith alan.coopersmith at oracle.com
Thu Oct 20 07:19:30 PDT 2011


On 10/20/11 03:44, Dave Airlie wrote:
> From: Dave Airlie<airlied at redhat.com>
>
> Pointed out by coverity scan.
>
> Signed-off-by: Dave Airlie<airlied at redhat.com>
> ---
>   hw/xfree86/common/xf86pciBus.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86pciBus.c b/hw/xfree86/common/xf86pciBus.c
> index eb5323c..21423ae 100644
> --- a/hw/xfree86/common/xf86pciBus.c
> +++ b/hw/xfree86/common/xf86pciBus.c
> @@ -657,7 +657,7 @@ xf86MatchPciInstances(const char *driverName, int vendorID,
>   	unsigned max_entries = numDevs;
>
>   	iter = pci_slot_match_iterator_create(NULL);
> -	while ((pPci = pci_device_next(iter)) != NULL) {
> +	while (pci_device_next(iter) != NULL) {
>   	    max_entries++;
>   	}
>

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list