Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Jun 2024 14:56:00 GMT
From:      Mariusz Zaborski <oshogbo@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 8aa9192ce98a - main - isp: fix ISPCTL_ABORT_CMD switch case
Message-ID:  <202406251456.45PEu0mQ051188@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by oshogbo:

URL: https://cgit.FreeBSD.org/src/commit/?id=8aa9192ce98aec07b24f8279c709237dd2c4421b

commit 8aa9192ce98aec07b24f8279c709237dd2c4421b
Author:     Mariusz Zaborski <oshogbo@FreeBSD.org>
AuthorDate: 2024-06-25 14:50:28 +0000
Commit:     Mariusz Zaborski <oshogbo@FreeBSD.org>
CommitDate: 2024-06-25 14:56:12 +0000

    isp: fix ISPCTL_ABORT_CMD switch case
    
    Prevent kernel panic by not running ISPCTL_FCLINK_TEST after a
    failed ISPCTL_ABORT_CMD.
    
    Reviewed by:    mav
    Tested by:      Arne Steinkamm <arne@steinkamm.com>
    MFC after:      3 days
    Differential Revision:  https://reviews.freebsd.org/D45718
---
 sys/dev/isp/isp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sys/dev/isp/isp.c b/sys/dev/isp/isp.c
index b064f86fda89..14d8147c3562 100644
--- a/sys/dev/isp/isp.c
+++ b/sys/dev/isp/isp.c
@@ -3099,6 +3099,7 @@ isp_control(ispsoftc_t *isp, ispctl_t ctl, ...)
 		if (ab->abrt_nphdl == ISP24XX_ABRT_OKAY)
 			return (0);
 		isp_prt(isp, ISP_LOGWARN, "Chan %d handle %d abort returned 0x%x", chan, tgt, ab->abrt_nphdl);
+		break;
 	}
 	case ISPCTL_FCLINK_TEST:
 	{



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