Skip site navigation (1)Skip section navigation (2) 
Date:      Mon, 16 Nov 2020 16:53:47 +0000 (UTC)
From:      Mark Johnston <markj@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r367727 - stable/12/sys/dev/qat
Message-ID:  <202011161653.0AGGrllo005442@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: markj
Date: Mon Nov 16 16:53:46 2020
New Revision: 367727
URL: https://svnweb.freebsd.org/changeset/base/367727

Log:
  MFC r367617:
  qat: Fix nits reported by Coverity

Modified:
  stable/12/sys/dev/qat/qat.c
  stable/12/sys/dev/qat/qat_ae.c
  stable/12/sys/dev/qat/qatvar.h
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sys/dev/qat/qat.c
==============================================================================
--- stable/12/sys/dev/qat/qat.c	Mon Nov 16 15:37:09 2020	(r367726)
+++ stable/12/sys/dev/qat/qat.c	Mon Nov 16 16:53:46 2020	(r367727)
@@ -573,7 +573,8 @@ qat_detach(device_t dev)
 		sc->sc_ih_cookie = NULL;
 	}
 	if (sc->sc_ih != NULL) {
-		(void)bus_release_resource(dev, SYS_RES_IRQ, i + 1, sc->sc_ih);
+		(void)bus_release_resource(dev, SYS_RES_IRQ,
+		    sc->sc_hw.qhw_num_banks + 1, sc->sc_ih);
 		sc->sc_ih = NULL;
 	}
 	pci_release_msi(dev);

Modified: stable/12/sys/dev/qat/qat_ae.c
==============================================================================
--- stable/12/sys/dev/qat/qat_ae.c	Mon Nov 16 15:37:09 2020	(r367726)
+++ stable/12/sys/dev/qat/qat_ae.c	Mon Nov 16 16:53:46 2020	(r367727)
@@ -2616,7 +2616,7 @@ qat_aefw_suof_load(struct qat_softc *sc, struct qat_dm
 static int
 qat_aefw_suof_write(struct qat_softc *sc)
 {
-	struct qat_suof_image *qsi = NULL;
+	struct qat_suof_image *qsi;
 	int i, error = 0;
 
 	for (i = 0; i < sc->sc_aefw_suof.qafs_num_simgs; i++) {
@@ -2626,21 +2626,20 @@ qat_aefw_suof_write(struct qat_softc *sc)
 		if (error)
 			return error;
 		error = qat_aefw_auth(sc, &qsi->qsi_dma);
-		if (error)
-			goto fail;
+		if (error) {
+			qat_free_dmamem(sc, &qsi->qsi_dma);
+			return error;
+		}
 		error = qat_aefw_suof_load(sc, &qsi->qsi_dma);
-		if (error)
-			goto fail;
-
+		if (error) {
+			qat_free_dmamem(sc, &qsi->qsi_dma);
+			return error;
+		}
 		qat_free_dmamem(sc, &qsi->qsi_dma);
 	}
 	qat_free_mem(sc->sc_aefw_suof.qafs_simg);
 
 	return 0;
-fail:
-	if (qsi != NULL)
-		qat_free_dmamem(sc, &qsi->qsi_dma);
-	return error;
 }
 
 static int

Modified: stable/12/sys/dev/qat/qatvar.h
==============================================================================
--- stable/12/sys/dev/qat/qatvar.h	Mon Nov 16 15:37:09 2020	(r367726)
+++ stable/12/sys/dev/qat/qatvar.h	Mon Nov 16 16:53:46 2020	(r367727)
@@ -596,7 +596,7 @@ struct qat_crypto_desc {
 	bus_addr_t qcd_desc_paddr;
 	bus_addr_t qcd_hash_state_paddr;
 
-	enum fw_slice qcd_slices[MAX_FW_SLICE];
+	enum fw_slice qcd_slices[MAX_FW_SLICE + 1];
 	enum fw_la_cmd_id qcd_cmd_id;
 	enum hw_cipher_dir qcd_cipher_dir;

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202011161653.0AGGrllo005442>