Merge ~paelzer/ubuntu/+source/libiscsi:fix-FTBFS-1894129-GROOVY into ubuntu/+source/libiscsi:ubuntu/groovy-devel

Proposed by Christian Ehrhardt 
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: d2755eb1d7db1a44da9a35b5c71c87bf06c2eaac
Merged at revision: d2755eb1d7db1a44da9a35b5c71c87bf06c2eaac
Proposed branch: ~paelzer/ubuntu/+source/libiscsi:fix-FTBFS-1894129-GROOVY
Merge into: ubuntu/+source/libiscsi:ubuntu/groovy-devel
Diff against target: 989 lines (+889/-1)
15 files modified
debian/changelog (+6/-0)
debian/control (+2/-1)
debian/patches/lp-1894129-test-tool-BlockEraseReserved-override-driver-queue_p.patch (+74/-0)
debian/patches/lp-1894129-test-tool-BlockEraseReserved-skip-unless-iSCSI.patch (+39/-0)
debian/patches/lp-1894129-test-tool-Compare-Write-override-driver-queue_pdu-ca.patch (+71/-0)
debian/patches/lp-1894129-test-tool-Compare-Write-skip-InvalidDataOutSize-unle.patch (+39/-0)
debian/patches/lp-1894129-test-tool-CryptoEraseReserved-override-driver-queue_.patch (+74/-0)
debian/patches/lp-1894129-test-tool-CryptoEraseReserved-skip-unless-iSCSI.patch (+39/-0)
debian/patches/lp-1894129-test-tool-OverwriteReserved-override-driver-queue_pd.patch (+68/-0)
debian/patches/lp-1894129-test-tool-Use-extern-int-in-headers.patch (+58/-0)
debian/patches/lp-1894129-test-tool-iSCSICmdSnTooHigh-override-driver-queue_pd.patch (+68/-0)
debian/patches/lp-1894129-test-tool-iSCSICmdSnTooLow-override-driver-queue_pdu.patch (+69/-0)
debian/patches/lp-1894129-test-tool-iSCSIDataSnInvalid-override-driver-queue_p.patch (+166/-0)
debian/patches/lp-1894129-test-tool-remove-unused-iscsi_queue_pdu-symbol-overl.patch (+104/-0)
debian/patches/series (+12/-0)
Reviewer Review Type Date Requested Status
Lucas Kanashiro (community) Approve
Canonical Server Pending
Canonical Server packageset reviewers Pending
Review via email: mp+390397@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

LGTM, +1.

review: Approve
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Thanks!

 * [new tag] upload/1.19.0-2ubuntu1 -> upload/1.19.0-2ubuntu1

Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading libiscsi_1.19.0-2ubuntu1.dsc: done.
  Uploading libiscsi_1.19.0-2ubuntu1.debian.tar.xz: done.
  Uploading libiscsi_1.19.0-2ubuntu1_source.buildinfo: done.
  Uploading libiscsi_1.19.0-2ubuntu1_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/debian/changelog b/debian/changelog
index 096d70b..bf25cdc 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
1libiscsi (1.19.0-2ubuntu1) groovy; urgency=medium
2
3 * d/p/lp-1894129-*: fix FTBFS (LP: #1894129)
4
5 -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Tue, 08 Sep 2020 14:31:03 +0200
6
1libiscsi (1.19.0-2) unstable; urgency=medium7libiscsi (1.19.0-2) unstable; urgency=medium
28
3 * attribute_packed.patch (Closes: #966864)9 * attribute_packed.patch (Closes: #966864)
diff --git a/debian/control b/debian/control
index 9310439..50f39ed 100644
--- a/debian/control
+++ b/debian/control
@@ -1,7 +1,8 @@
1Source: libiscsi1Source: libiscsi
2Section: net2Section: net
3Priority: optional3Priority: optional
4Maintainer: Michael Tokarev <mjt@tls.msk.ru>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Michael Tokarev <mjt@tls.msk.ru>
5Build-Depends: debhelper-compat (= 12), libcunit1-dev, librdmacm-dev6Build-Depends: debhelper-compat (= 12), libcunit1-dev, librdmacm-dev
6Rules-Requires-Root: no7Rules-Requires-Root: no
7Standards-Version: 4.5.08Standards-Version: 4.5.0
diff --git a/debian/patches/lp-1894129-test-tool-BlockEraseReserved-override-driver-queue_p.patch b/debian/patches/lp-1894129-test-tool-BlockEraseReserved-override-driver-queue_p.patch
8new file mode 1006449new file mode 100644
index 0000000..68def73
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-BlockEraseReserved-override-driver-queue_p.patch
@@ -0,0 +1,74 @@
1From b705f9e5240583fb862f16d603e5cf2120e5d54c Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/BlockEraseReserved: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10
11Signed-off-by: David Disseldorp <ddiss@suse.de>
12
13Origin: upstream, https://github.com/sahlberg/libiscsi/commit/b705f9e5
14Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
15Last-Update: 2020-09-08
16
17---
18 test-tool/test_sanitize_block_erase_reserved.c | 17 +++++++++++------
19 1 file changed, 11 insertions(+), 6 deletions(-)
20
21diff --git a/test-tool/test_sanitize_block_erase_reserved.c b/test-tool/test_sanitize_block_erase_reserved.c
22index fcb862e..4a0d2cb 100644
23--- a/test-tool/test_sanitize_block_erase_reserved.c
24+++ b/test-tool/test_sanitize_block_erase_reserved.c
25@@ -25,8 +25,9 @@
26 #include "iscsi-test-cu.h"
27
28 static int change_num;
29+static struct iscsi_transport iscsi_drv_orig;
30
31-static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu *pdu)
32+static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
33 {
34 switch (change_num) {
35 case 1:
36@@ -43,12 +44,12 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu
37 break;
38 }
39
40- change_num = 0;
41- return 0;
42+ change_num = 0;
43+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
44 }
45
46 void test_sanitize_block_erase_reserved(void)
47-{
48+{
49 int i;
50
51 logging(LOG_VERBOSE, LOG_BLANK_LINE);
52@@ -64,8 +65,9 @@ void test_sanitize_block_erase_reserved(void)
53 return;
54 }
55
56- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
57-
58+ /* override transport queue_pdu callback for PDU manipulation */
59+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
60+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
61
62 logging(LOG_VERBOSE, "Send SANITIZE command with the reserved "
63 "bit in byte 1 set to 1");
64@@ -81,4 +83,7 @@ void test_sanitize_block_erase_reserved(void)
65 SANITIZE(sd, 0, 0, SCSI_SANITIZE_BLOCK_ERASE, 0, NULL,
66 EXPECT_INVALID_FIELD_IN_CDB);
67 }
68+
69+ /* restore transport callbacks */
70+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
71 }
72--
732.28.0
74
diff --git a/debian/patches/lp-1894129-test-tool-BlockEraseReserved-skip-unless-iSCSI.patch b/debian/patches/lp-1894129-test-tool-BlockEraseReserved-skip-unless-iSCSI.patch
0new file mode 10064475new file mode 100644
index 0000000..43ae51b
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-BlockEraseReserved-skip-unless-iSCSI.patch
@@ -0,0 +1,39 @@
1From fdc45fadfa06f2543a12abe8eefe7f87d5dab0c2 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 15:40:00 +0200
4Subject: [PATCH] test-tool/BlockEraseReserved: skip unless iSCSI
5
6This test performs PDU manipulation so should only be run against iSCSI
7devices.
8
9Signed-off-by: David Disseldorp <ddiss@suse.de>
10
11Origin: upstream, https://github.com/sahlberg/libiscsi/commit/fdc45fad
12Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
13Last-Update: 2020-09-08
14
15---
16 test-tool/test_sanitize_block_erase_reserved.c | 7 +++++++
17 1 file changed, 7 insertions(+)
18
19diff --git a/test-tool/test_sanitize_block_erase_reserved.c b/test-tool/test_sanitize_block_erase_reserved.c
20index 55c85d9..fcb862e 100644
21--- a/test-tool/test_sanitize_block_erase_reserved.c
22+++ b/test-tool/test_sanitize_block_erase_reserved.c
23@@ -56,6 +56,13 @@ void test_sanitize_block_erase_reserved(void)
24
25 CHECK_FOR_SANITIZE;
26 CHECK_FOR_DATALOSS;
27+ if (sd->iscsi_ctx == NULL) {
28+ const char *err = "[SKIPPED] This test is "
29+ "only supported for iSCSI backends";
30+ logging(LOG_NORMAL, "%s", err);
31+ CU_PASS(err);
32+ return;
33+ }
34
35 local_iscsi_queue_pdu = my_iscsi_queue_pdu;
36
37--
382.28.0
39
diff --git a/debian/patches/lp-1894129-test-tool-Compare-Write-override-driver-queue_pdu-ca.patch b/debian/patches/lp-1894129-test-tool-Compare-Write-override-driver-queue_pdu-ca.patch
0new file mode 10064440new file mode 100644
index 0000000..d0433a2
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-Compare-Write-override-driver-queue_pdu-ca.patch
@@ -0,0 +1,71 @@
1From 310e9ace02bafb78d69bfe20e5c9befe7ed24a10 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/Compare&Write: override driver queue_pdu() callback
5
6In preparation for removing the test-tool iscsi_queue_pdu() dlsym
7overload hack, convert the Compare&Write InvalidDataOutSize test to use
8the regular transport driver callback for this purpose.
9
10Signed-off-by: David Disseldorp <ddiss@suse.de>
11
12Origin: upstream, https://github.com/sahlberg/libiscsi/commit/310e9ace
13Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
14Last-Update: 2020-09-08
15
16---
17 .../test_compareandwrite_invalid_dataout_size.c | 16 +++++++++++-----
18 1 file changed, 11 insertions(+), 5 deletions(-)
19
20diff --git a/test-tool/test_compareandwrite_invalid_dataout_size.c b/test-tool/test_compareandwrite_invalid_dataout_size.c
21index eb6a6dd..9fcd739 100644
22--- a/test-tool/test_compareandwrite_invalid_dataout_size.c
23+++ b/test-tool/test_compareandwrite_invalid_dataout_size.c
24@@ -27,11 +27,12 @@
25
26
27 static int new_tl;
28+static struct iscsi_transport iscsi_drv_orig;
29
30-static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu *pdu _U_)
31+static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
32 {
33 if (pdu->outdata.data[0] != ISCSI_PDU_SCSI_REQUEST) {
34- return 0;
35+ goto out;
36 }
37 switch (new_tl) {
38 case 1:
39@@ -43,7 +44,8 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu
40 pdu->outdata.data[32 + 13] = 3;
41 break;
42 }
43- return 0;
44+out:
45+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
46 }
47
48 void
49@@ -61,8 +63,10 @@ test_compareandwrite_invalid_dataout_size(void)
50 return;
51 }
52
53- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
54-
55+ /* override transport queue_pdu callback for PDU manipulation */
56+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
57+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
58+
59 logging(LOG_VERBOSE, LOG_BLANK_LINE);
60 logging(LOG_VERBOSE, "Test that COMPAREANDWRITE fails for invalid "
61 "(too small/too large) DataOut sizes.");
62@@ -89,4 +93,6 @@ test_compareandwrite_invalid_dataout_size(void)
63 block_size, 0, 0, 0, 0,
64 EXPECT_STATUS_GENERIC_BAD);
65
66+ /* restore transport callbacks */
67+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
68 }
69--
702.28.0
71
diff --git a/debian/patches/lp-1894129-test-tool-Compare-Write-skip-InvalidDataOutSize-unle.patch b/debian/patches/lp-1894129-test-tool-Compare-Write-skip-InvalidDataOutSize-unle.patch
0new file mode 10064472new file mode 100644
index 0000000..179a622
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-Compare-Write-skip-InvalidDataOutSize-unle.patch
@@ -0,0 +1,39 @@
1From 60a5856f00731be578a57bfbf68b2e9858680f07 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:33:50 +0200
4Subject: [PATCH] test-tool/Compare&Write: skip InvalidDataOutSize unless iSCSI
5
6This test performs PDU manipulation so should only be run against iSCSI
7devices.
8
9Signed-off-by: David Disseldorp <ddiss@suse.de>
10
11Origin: upstream, https://github.com/sahlberg/libiscsi/commit/60a5856f
12Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
13Last-Update: 2020-09-08
14
15---
16 test-tool/test_compareandwrite_invalid_dataout_size.c | 7 +++++++
17 1 file changed, 7 insertions(+)
18
19diff --git a/test-tool/test_compareandwrite_invalid_dataout_size.c b/test-tool/test_compareandwrite_invalid_dataout_size.c
20index 4abd501..eb6a6dd 100644
21--- a/test-tool/test_compareandwrite_invalid_dataout_size.c
22+++ b/test-tool/test_compareandwrite_invalid_dataout_size.c
23@@ -53,6 +53,13 @@ test_compareandwrite_invalid_dataout_size(void)
24 CHECK_FOR_THIN_PROVISIONING;
25 CHECK_FOR_LBPPB_GT_1;
26 CHECK_FOR_SBC;
27+ if (sd->iscsi_ctx == NULL) {
28+ const char *err = "[SKIPPED] This test is "
29+ "only supported for iSCSI backends";
30+ logging(LOG_NORMAL, "%s", err);
31+ CU_PASS(err);
32+ return;
33+ }
34
35 local_iscsi_queue_pdu = my_iscsi_queue_pdu;
36
37--
382.28.0
39
diff --git a/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-override-driver-queue_.patch b/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-override-driver-queue_.patch
0new file mode 10064440new file mode 100644
index 0000000..0c4a300
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-override-driver-queue_.patch
@@ -0,0 +1,74 @@
1From 77279dd44fd19167c22cfada2eec377058e1bde9 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/CryptoEraseReserved: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10
11Signed-off-by: David Disseldorp <ddiss@suse.de>
12
13Origin: upstream, https://github.com/sahlberg/libiscsi/commit/77279dd4
14Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
15Last-Update: 2020-09-08
16
17---
18 test-tool/test_sanitize_crypto_erase_reserved.c | 17 +++++++++++------
19 1 file changed, 11 insertions(+), 6 deletions(-)
20
21diff --git a/test-tool/test_sanitize_crypto_erase_reserved.c b/test-tool/test_sanitize_crypto_erase_reserved.c
22index 00bbf34..646cbea 100644
23--- a/test-tool/test_sanitize_crypto_erase_reserved.c
24+++ b/test-tool/test_sanitize_crypto_erase_reserved.c
25@@ -25,8 +25,9 @@
26 #include "iscsi-test-cu.h"
27
28 static int change_num;
29+static struct iscsi_transport iscsi_drv_orig;
30
31-static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu *pdu)
32+static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
33 {
34 switch (change_num) {
35 case 1:
36@@ -43,12 +44,12 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu
37 break;
38 }
39
40- change_num = 0;
41- return 0;
42+ change_num = 0;
43+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
44 }
45
46 void test_sanitize_crypto_erase_reserved(void)
47-{
48+{
49 int i;
50
51 logging(LOG_VERBOSE, LOG_BLANK_LINE);
52@@ -64,8 +65,9 @@ void test_sanitize_crypto_erase_reserved(void)
53 return;
54 }
55
56- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
57-
58+ /* override transport queue_pdu callback for PDU manipulation */
59+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
60+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
61
62 logging(LOG_VERBOSE, "Send SANITIZE command with the reserved "
63 "bit in byte 1 set to 1");
64@@ -81,4 +83,7 @@ void test_sanitize_crypto_erase_reserved(void)
65 SANITIZE(sd, 0, 0, SCSI_SANITIZE_CRYPTO_ERASE, 0, NULL,
66 EXPECT_INVALID_FIELD_IN_CDB);
67 }
68+
69+ /* restore transport callbacks */
70+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
71 }
72--
732.28.0
74
diff --git a/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-skip-unless-iSCSI.patch b/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-skip-unless-iSCSI.patch
0new file mode 10064475new file mode 100644
index 0000000..95ca3a3
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-CryptoEraseReserved-skip-unless-iSCSI.patch
@@ -0,0 +1,39 @@
1From c830a3b55e91fe8b1cb97e6899ef872691c8804e Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 15:40:00 +0200
4Subject: [PATCH] test-tool/CryptoEraseReserved: skip unless iSCSI
5
6This test performs PDU manipulation so should only be run against iSCSI
7devices.
8
9Signed-off-by: David Disseldorp <ddiss@suse.de>
10
11Origin: upstream, https://github.com/sahlberg/libiscsi/commit/c830a3b5
12Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
13Last-Update: 2020-09-08
14
15---
16 test-tool/test_sanitize_crypto_erase_reserved.c | 7 +++++++
17 1 file changed, 7 insertions(+)
18
19diff --git a/test-tool/test_sanitize_crypto_erase_reserved.c b/test-tool/test_sanitize_crypto_erase_reserved.c
20index c713ab5..00bbf34 100644
21--- a/test-tool/test_sanitize_crypto_erase_reserved.c
22+++ b/test-tool/test_sanitize_crypto_erase_reserved.c
23@@ -56,6 +56,13 @@ void test_sanitize_crypto_erase_reserved(void)
24
25 CHECK_FOR_SANITIZE;
26 CHECK_FOR_DATALOSS;
27+ if (sd->iscsi_ctx == NULL) {
28+ const char *err = "[SKIPPED] This test is "
29+ "only supported for iSCSI backends";
30+ logging(LOG_NORMAL, "%s", err);
31+ CU_PASS(err);
32+ return;
33+ }
34
35 local_iscsi_queue_pdu = my_iscsi_queue_pdu;
36
37--
382.28.0
39
diff --git a/debian/patches/lp-1894129-test-tool-OverwriteReserved-override-driver-queue_pd.patch b/debian/patches/lp-1894129-test-tool-OverwriteReserved-override-driver-queue_pd.patch
0new file mode 10064440new file mode 100644
index 0000000..0b6e27b
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-OverwriteReserved-override-driver-queue_pd.patch
@@ -0,0 +1,68 @@
1From 2ae45ac15d8c1d4a4f68a085dadb7f43e2f2e280 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/OverwriteReserved: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10
11Signed-off-by: David Disseldorp <ddiss@suse.de>
12
13Origin: backport, https://github.com/sahlberg/libiscsi/commit/2ae45ac1
14Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
15Last-Update: 2020-09-08
16
17---
18 test-tool/test_sanitize_overwrite_reserved.c | 16 +++++++++++-----
19 1 file changed, 11 insertions(+), 5 deletions(-)
20
21--- a/test-tool/test_sanitize_overwrite_reserved.c
22+++ b/test-tool/test_sanitize_overwrite_reserved.c
23@@ -26,8 +26,9 @@
24 #include "iscsi-test-cu.h"
25
26 static int change_num;
27+static struct iscsi_transport iscsi_drv_orig;
28
29-static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu *pdu)
30+static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
31 {
32 switch (change_num) {
33 case 1:
34@@ -44,12 +45,12 @@ static int my_iscsi_queue_pdu(struct isc
35 break;
36 }
37
38- change_num = 0;
39- return 0;
40+ change_num = 0;
41+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
42 }
43
44 void test_sanitize_overwrite_reserved(void)
45-{
46+{
47 int i;
48 struct iscsi_data data;
49
50@@ -68,7 +69,9 @@ void test_sanitize_overwrite_reserved(vo
51 CHECK_FOR_SANITIZE;
52 CHECK_FOR_DATALOSS;
53
54- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
55+ /* override transport queue_pdu callback for PDU manipulation */
56+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
57+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
58
59 logging(LOG_VERBOSE, "Send SANITIZE command with the reserved "
60 "bit in byte 1 set to 1");
61@@ -84,4 +87,7 @@ void test_sanitize_overwrite_reserved(vo
62 SANITIZE(sd, 0, 0, SCSI_SANITIZE_OVERWRITE, data.size, &data,
63 EXPECT_INVALID_FIELD_IN_CDB);
64 }
65+
66+ /* restore transport callbacks */
67+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
68 }
diff --git a/debian/patches/lp-1894129-test-tool-Use-extern-int-in-headers.patch b/debian/patches/lp-1894129-test-tool-Use-extern-int-in-headers.patch
0new file mode 10064469new file mode 100644
index 0000000..f10a7a4
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-Use-extern-int-in-headers.patch
@@ -0,0 +1,58 @@
1From ea89dcdbf167acd634300296172926c2e8d85804 Mon Sep 17 00:00:00 2001
2From: Cole Robinson <crobinso@redhat.com>
3Date: Sat, 15 Feb 2020 18:03:11 -0500
4Subject: [PATCH] test-tool: Use 'extern int' in headers
5
6Without this, linking fails on Fedora rawhide. Example:
7
8...
9/usr/bin/ld: test_async_lu_reset_simple.o:/root/libiscsi/libiscsi.git/test-tool/iscsi-support.h:59: multiple definition of `param_list_len_err_ascqs'; iscsi-test-cu.o:/root/libiscsi/libiscsi.git/test-tool/iscsi-support.h:59: first defined here
10/usr/bin/ld: test_async_lu_reset_simple.o:/root/libiscsi/libiscsi.git/test-tool/iscsi-support.h:58: multiple definition of `invalid_cdb_ascqs'; iscsi-test-cu.o:/root/libiscsi/libiscsi.git/test-tool/iscsi-support.h:58: first defined here
11...
12
13Signed-off-by: Cole Robinson <crobinso@redhat.com>
14
15Origin: upstream, https://github.com/sahlberg/libiscsi/commit/ea89dcdb
16Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
17Last-Update: 2020-09-08
18
19---
20 test-tool/iscsi-support.h | 22 +++++++++++-----------
21 1 file changed, 11 insertions(+), 11 deletions(-)
22
23diff --git a/test-tool/iscsi-support.h b/test-tool/iscsi-support.h
24index 7aa9e45..9c3a461 100644
25--- a/test-tool/iscsi-support.h
26+++ b/test-tool/iscsi-support.h
27@@ -53,17 +53,17 @@ extern const char *initiatorname2;
28 #define EXPECT_RESERVATION_CONFLICT SCSI_STATUS_RESERVATION_CONFLICT, 0, NULL, 0
29 #define EXPECT_COPY_ABORTED SCSI_STATUS_CHECK_CONDITION, SCSI_SENSE_COPY_ABORTED, copy_aborted_ascqs, 3
30
31-int no_medium_ascqs[3];
32-int lba_oob_ascqs[1];
33-int invalid_cdb_ascqs[2];
34-int param_list_len_err_ascqs[1];
35-int too_many_desc_ascqs[2];
36-int unsupp_desc_code_ascqs[2];
37-int write_protect_ascqs[3];
38-int sanitize_ascqs[1];
39-int removal_ascqs[1];
40-int miscompare_ascqs[1];
41-int copy_aborted_ascqs[3];
42+extern int no_medium_ascqs[3];
43+extern int lba_oob_ascqs[1];
44+extern int invalid_cdb_ascqs[2];
45+extern int param_list_len_err_ascqs[1];
46+extern int too_many_desc_ascqs[2];
47+extern int unsupp_desc_code_ascqs[2];
48+extern int write_protect_ascqs[3];
49+extern int sanitize_ascqs[1];
50+extern int removal_ascqs[1];
51+extern int miscompare_ascqs[1];
52+extern int copy_aborted_ascqs[3];
53
54 extern int loglevel;
55 #define LOG_SILENT 0
56--
572.28.0
58
diff --git a/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooHigh-override-driver-queue_pd.patch b/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooHigh-override-driver-queue_pd.patch
0new file mode 10064459new file mode 100644
index 0000000..5df90d9
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooHigh-override-driver-queue_pd.patch
@@ -0,0 +1,68 @@
1From 575ba2244c5db240bc29fa493404eee39f693a2d Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/iSCSICmdSnTooHigh: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10
11Signed-off-by: David Disseldorp <ddiss@suse.de>
12
13Origin: upstream, https://github.com/sahlberg/libiscsi/commit/575ba224
14Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
15Last-Update: 2020-09-08
16
17---
18 test-tool/test_iscsi_cmdsn_toohigh.c | 11 ++++++++---
19 1 file changed, 8 insertions(+), 3 deletions(-)
20
21diff --git a/test-tool/test_iscsi_cmdsn_toohigh.c b/test-tool/test_iscsi_cmdsn_toohigh.c
22index 59ce0ab..19c8b91 100644
23--- a/test-tool/test_iscsi_cmdsn_toohigh.c
24+++ b/test-tool/test_iscsi_cmdsn_toohigh.c
25@@ -25,6 +25,7 @@
26 #include "iscsi-test-cu.h"
27
28 static int change_cmdsn;
29+static struct iscsi_transport iscsi_drv_orig;
30
31 static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
32 {
33@@ -40,7 +41,7 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu
34 }
35
36 change_cmdsn = 0;
37- return 0;
38+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
39 }
40
41 void test_iscsi_cmdsn_toohigh(void)
42@@ -64,7 +65,9 @@ void test_iscsi_cmdsn_toohigh(void)
43
44 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
45 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
46- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
47+ /* override transport queue_pdu callback for PDU manipulation */
48+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
49+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
50 change_cmdsn = 1;
51 /* we don't want autoreconnect since some targets will incorrectly
52 * drop the connection on this condition.
53@@ -81,9 +84,11 @@ void test_iscsi_cmdsn_toohigh(void)
54 logging(LOG_VERBOSE, "[FAILURE] We got a response from the target but SMDSN was outside of the window.");
55 }
56
57-
58 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
59 logging(LOG_VERBOSE, "Send a TESTUNITREADY with CMDSN == EXPCMDSN. should work again");
60 TESTUNITREADY(sd,
61 EXPECT_STATUS_GOOD);
62+
63+ /* restore transport callbacks */
64+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
65 }
66--
672.28.0
68
diff --git a/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooLow-override-driver-queue_pdu.patch b/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooLow-override-driver-queue_pdu.patch
0new file mode 10064469new file mode 100644
index 0000000..4dcd73d
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-iSCSICmdSnTooLow-override-driver-queue_pdu.patch
@@ -0,0 +1,69 @@
1From c067eef9c10c17b8a9be83ee4f71d4c8d0d238e4 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/iSCSICmdSnTooLow: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10
11Signed-off-by: David Disseldorp <ddiss@suse.de>
12
13Origin: upstream, https://github.com/sahlberg/libiscsi/commit/c067eef9
14Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
15Last-Update: 2020-09-08
16
17---
18 test-tool/test_iscsi_cmdsn_toolow.c | 12 ++++++++----
19 1 file changed, 8 insertions(+), 4 deletions(-)
20
21diff --git a/test-tool/test_iscsi_cmdsn_toolow.c b/test-tool/test_iscsi_cmdsn_toolow.c
22index 0b980b3..23b93c3 100644
23--- a/test-tool/test_iscsi_cmdsn_toolow.c
24+++ b/test-tool/test_iscsi_cmdsn_toolow.c
25@@ -25,6 +25,7 @@
26 #include "iscsi-test-cu.h"
27
28 static int change_cmdsn;
29+static struct iscsi_transport iscsi_drv_orig;
30
31 static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
32 {
33@@ -40,7 +41,7 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu
34 }
35
36 change_cmdsn = 0;
37- return 0;
38+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
39 }
40
41 void test_iscsi_cmdsn_toolow(void)
42@@ -64,7 +65,9 @@ void test_iscsi_cmdsn_toolow(void)
43
44 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
45 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
46- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
47+ /* override transport queue_pdu callback for PDU manipulation */
48+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
49+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
50 change_cmdsn = 1;
51 /* we don't want autoreconnect since some targets will incorrectly
52 * drop the connection on this condition.
53@@ -81,10 +84,11 @@ void test_iscsi_cmdsn_toolow(void)
54 logging(LOG_VERBOSE, "[FAILURE] We got a response from the target but SMDSN was outside of the window.");
55 }
56
57-
58-
59 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
60 logging(LOG_VERBOSE, "Send a TESTUNITREADY with CMDSN == EXPCMDSN. should work again");
61 TESTUNITREADY(sd,
62 EXPECT_STATUS_GOOD);
63+
64+ /* restore transport callbacks */
65+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
66 }
67--
682.28.0
69
diff --git a/debian/patches/lp-1894129-test-tool-iSCSIDataSnInvalid-override-driver-queue_p.patch b/debian/patches/lp-1894129-test-tool-iSCSIDataSnInvalid-override-driver-queue_p.patch
0new file mode 10064470new file mode 100644
index 0000000..4a253a1
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-iSCSIDataSnInvalid-override-driver-queue_p.patch
@@ -0,0 +1,166 @@
1From 153ca193fa61be8c2c26d8736119e7dc873d1242 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 14:46:28 +0200
4Subject: [PATCH] test-tool/iSCSIDataSnInvalid: override driver queue_pdu()
5 callback
6
7In preparation for removing the test-tool iscsi_queue_pdu() dlsym
8overload hack, convert this test to use the regular transport driver
9callback for this purpose.
10This commit also removes a number of unnecessary "WRITE10 is not
11implemented" checks, which should all be caught by the first check.
12
13Signed-off-by: David Disseldorp <ddiss@suse.de>
14
15Origin: upstream, https://github.com/sahlberg/libiscsi/commit/153ca193
16Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
17Last-Update: 2020-09-08
18
19---
20 test-tool/test_iscsi_datasn_invalid.c | 56 ++++++++++-----------------
21 1 file changed, 21 insertions(+), 35 deletions(-)
22
23diff --git a/test-tool/test_iscsi_datasn_invalid.c b/test-tool/test_iscsi_datasn_invalid.c
24index 14deb7a..c25631a 100644
25--- a/test-tool/test_iscsi_datasn_invalid.c
26+++ b/test-tool/test_iscsi_datasn_invalid.c
27@@ -25,13 +25,14 @@
28 #include "iscsi-test-cu.h"
29
30 static int change_datasn;
31+static struct iscsi_transport iscsi_drv_orig;
32
33-static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu *pdu _U_)
34+static int my_iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
35 {
36 uint32_t datasn;
37
38 if (pdu->outdata.data[0] != ISCSI_PDU_DATA_OUT) {
39- return 0;
40+ goto out;
41 }
42 switch (change_datasn) {
43 case 1:
44@@ -52,11 +53,12 @@ static int my_iscsi_queue_pdu(struct iscsi_context *iscsi _U_, struct iscsi_pdu
45 scsi_set_uint32(&pdu->outdata.data[36], 1 - datasn);
46 break;
47 }
48- return 0;
49+out:
50+ return iscsi_drv_orig.queue_pdu(iscsi, pdu);
51 }
52
53 void test_iscsi_datasn_invalid(void)
54-{
55+{
56 int ret;
57
58 CHECK_FOR_DATALOSS;
59@@ -77,7 +79,9 @@ void test_iscsi_datasn_invalid(void)
60
61 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
62 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
63- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
64+ /* override transport queue_pdu callback for PDU manipulation */
65+ iscsi_drv_orig = *sd->iscsi_ctx->drv;
66+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
67 iscsi_set_noautoreconnect(sd->iscsi_ctx, 1);
68 iscsi_set_timeout(sd->iscsi_ctx, 3);
69
70@@ -89,81 +93,63 @@ void test_iscsi_datasn_invalid(void)
71 if (ret == -2) {
72 logging(LOG_NORMAL, "[SKIPPED] WRITE10 is not implemented.");
73 CU_PASS("WRITE10 is not implemented.");
74- local_iscsi_queue_pdu = NULL;
75- return;
76- }
77+ goto out_ctx_restore;
78+ }
79 CU_ASSERT_NOT_EQUAL(ret, 0);
80
81+ /* avoid changing DataSN during reconnect */
82+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
83 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
84
85-
86 logging(LOG_VERBOSE, "Send Data-Out PDU with DataSN==27. Should fail");
87 change_datasn = 2;
88
89 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
90 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
91- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
92+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
93 iscsi_set_noautoreconnect(sd->iscsi_ctx, 1);
94 iscsi_set_timeout(sd->iscsi_ctx, 3);
95
96 ret = write10(sd, 100, block_size,
97 block_size, 0, 0, 0, 0, 0, scratch,
98 EXPECT_STATUS_GOOD);
99- if (ret == -2) {
100- logging(LOG_NORMAL, "[SKIPPED] WRITE10 is not implemented.");
101- CU_PASS("WRITE10 is not implemented.");
102- local_iscsi_queue_pdu = NULL;
103- return;
104- }
105 CU_ASSERT_NOT_EQUAL(ret, 0);
106
107+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
108 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
109
110-
111 logging(LOG_VERBOSE, "Send Data-Out PDU with DataSN==-1. Should fail");
112 change_datasn = 3;
113
114 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
115 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
116- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
117+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
118 iscsi_set_noautoreconnect(sd->iscsi_ctx, 1);
119 iscsi_set_timeout(sd->iscsi_ctx, 3);
120
121 ret = write10(sd, 100, block_size,
122 block_size, 0, 0, 0, 0, 0, scratch,
123 EXPECT_STATUS_GOOD);
124- if (ret == -2) {
125- logging(LOG_NORMAL, "[SKIPPED] WRITE10 is not implemented.");
126- CU_PASS("WRITE10 is not implemented.");
127- local_iscsi_queue_pdu = NULL;
128- return;
129- }
130 CU_ASSERT_NOT_EQUAL(ret, 0);
131
132+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
133 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
134
135-
136-
137 logging(LOG_VERBOSE, "Send Data-Out PDU's in reverse order (DataSN == 1,0). Should fail");
138 change_datasn = 4;
139
140 sd->iscsi_ctx->use_immediate_data = ISCSI_IMMEDIATE_DATA_NO;
141 sd->iscsi_ctx->target_max_recv_data_segment_length = block_size;
142- local_iscsi_queue_pdu = my_iscsi_queue_pdu;
143+ sd->iscsi_ctx->drv->queue_pdu = my_iscsi_queue_pdu;
144 iscsi_set_noautoreconnect(sd->iscsi_ctx, 1);
145 iscsi_set_timeout(sd->iscsi_ctx, 3);
146
147 ret = write10(sd, 100, 2 * block_size,
148 block_size, 0, 0, 0, 0, 0, scratch,
149 EXPECT_STATUS_GOOD);
150- if (ret == -2) {
151- logging(LOG_NORMAL, "[SKIPPED] WRITE10 is not implemented.");
152- CU_PASS("WRITE10 is not implemented.");
153- local_iscsi_queue_pdu = NULL;
154- return;
155- }
156 CU_ASSERT_NOT_EQUAL(ret, 0);
157-
158- local_iscsi_queue_pdu = NULL;
159+out_ctx_restore:
160+ /* restore transport callbacks and autoreconnect */
161+ *(sd->iscsi_ctx->drv) = iscsi_drv_orig;
162 iscsi_set_noautoreconnect(sd->iscsi_ctx, 0);
163 }
164--
1652.28.0
166
diff --git a/debian/patches/lp-1894129-test-tool-remove-unused-iscsi_queue_pdu-symbol-overl.patch b/debian/patches/lp-1894129-test-tool-remove-unused-iscsi_queue_pdu-symbol-overl.patch
0new file mode 100644167new file mode 100644
index 0000000..6cd1a9f
--- /dev/null
+++ b/debian/patches/lp-1894129-test-tool-remove-unused-iscsi_queue_pdu-symbol-overl.patch
@@ -0,0 +1,104 @@
1From dd17293bcc188dbefc420ee23b77bc9bb7c5e9a2 Mon Sep 17 00:00:00 2001
2From: David Disseldorp <ddiss@suse.de>
3Date: Mon, 16 Sep 2019 16:02:36 +0200
4Subject: [PATCH] test-tool: remove unused iscsi_queue_pdu() symbol overload
5
6All users have now been changed to overload via the iscsi_ctx transport
7callback, so the dlsym based overloading can now be dropped.
8
9Signed-off-by: David Disseldorp <ddiss@suse.de>
10
11Origin: upstream, https://github.com/sahlberg/libiscsi/commit/dd17293b
12Bug-Ubuntu: https://bugs.launchpad.net/bugs/1894129
13Last-Update: 2020-09-08
14
15---
16 test-tool/iscsi-support.c | 11 -----------
17 test-tool/iscsi-support.h | 1 -
18 test-tool/iscsi-test-cu.c | 15 ---------------
19 3 files changed, 27 deletions(-)
20
21diff --git a/test-tool/iscsi-support.c b/test-tool/iscsi-support.c
22index 4b49478..169d2b0 100644
23--- a/test-tool/iscsi-support.c
24+++ b/test-tool/iscsi-support.c
25@@ -119,8 +119,6 @@ int readonly;
26 int sbc3_support;
27 int maximum_transfer_length;
28
29-int (*real_iscsi_queue_pdu)(struct iscsi_context *iscsi, struct iscsi_pdu *pdu);
30-
31 static const unsigned char zeroBlock[4096];
32
33 /**
34@@ -569,15 +567,6 @@ wait_until_test_finished(struct iscsi_context *iscsi, struct iscsi_async_state *
35 }
36 }
37
38-int
39-iscsi_queue_pdu(struct iscsi_context *iscsi, struct iscsi_pdu *pdu)
40-{
41- if (local_iscsi_queue_pdu != NULL) {
42- local_iscsi_queue_pdu(iscsi, pdu);
43- }
44- return real_iscsi_queue_pdu(iscsi, pdu);
45-}
46-
47 int
48 orwrite(struct scsi_device *sdev, uint64_t lba,
49 uint32_t datalen, int blocksize, int wrprotect,
50diff --git a/test-tool/iscsi-support.h b/test-tool/iscsi-support.h
51index aee00ca..2b20a1f 100644
52--- a/test-tool/iscsi-support.h
53+++ b/test-tool/iscsi-support.h
54@@ -782,7 +782,6 @@ struct iscsi_async_state {
55 void wait_until_test_finished(struct iscsi_context *iscsi, struct iscsi_async_state *test_state);
56
57 struct iscsi_pdu;
58-int (*local_iscsi_queue_pdu)(struct iscsi_context *iscsi, struct iscsi_pdu *pdu);
59
60 struct scsi_command_descriptor *get_command_descriptor(int opcode, int sa);
61
62diff --git a/test-tool/iscsi-test-cu.c b/test-tool/iscsi-test-cu.c
63index eafda39..26e1b05 100644
64--- a/test-tool/iscsi-test-cu.c
65+++ b/test-tool/iscsi-test-cu.c
66@@ -59,12 +59,6 @@ int loglevel = LOG_NORMAL;
67 struct scsi_device *sd = NULL; /* mp_sds[0] alias */
68 static unsigned int maxsectors;
69
70-/*
71- * this allows us to redefine how PDU are queued, at times, for
72- * testing purposes
73- */
74-int (*real_iscsi_queue_pdu)(struct iscsi_context *iscsi, struct iscsi_pdu *pdu);
75-
76 /*****************************************************************
77 *
78 * list of tests and test suites
79@@ -805,7 +799,6 @@ test_setup(void)
80 {
81 task = NULL;
82 read_write_buf = NULL;
83- local_iscsi_queue_pdu = NULL;
84 }
85
86 void
87@@ -1214,14 +1207,6 @@ main(int argc, char *argv[])
88 mp_num_sds++;
89 }
90
91- /* So that we can override iscsi_queue_pdu in tests
92- * and replace or mutate the blob that we are about to write to the
93- * wire.
94- * This allows such tests to do their mutates and then call out
95- * to the real queueing function once they have modified the data.
96- */
97- real_iscsi_queue_pdu = dlsym(RTLD_NEXT, "iscsi_queue_pdu");
98-
99 if ((mp_num_sds == 0) || (mp_sds[0]->iscsi_url == NULL
100 && mp_sds[0]->sgio_dev == NULL)) {
101 #ifdef HAVE_SG_IO
102--
1032.28.0
104
diff --git a/debian/patches/series b/debian/patches/series
index de65f63..36e1d44 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -3,3 +3,15 @@ soversion.diff
3libpthread.diff3libpthread.diff
4attribute_packed.patch4attribute_packed.patch
5spelling.patch5spelling.patch
6lp-1894129-test-tool-Compare-Write-skip-InvalidDataOutSize-unle.patch
7lp-1894129-test-tool-Compare-Write-override-driver-queue_pdu-ca.patch
8lp-1894129-test-tool-iSCSICmdSnTooLow-override-driver-queue_pdu.patch
9lp-1894129-test-tool-iSCSICmdSnTooHigh-override-driver-queue_pd.patch
10lp-1894129-test-tool-iSCSIDataSnInvalid-override-driver-queue_p.patch
11lp-1894129-test-tool-BlockEraseReserved-skip-unless-iSCSI.patch
12lp-1894129-test-tool-CryptoEraseReserved-skip-unless-iSCSI.patch
13lp-1894129-test-tool-CryptoEraseReserved-override-driver-queue_.patch
14lp-1894129-test-tool-OverwriteReserved-override-driver-queue_pd.patch
15lp-1894129-test-tool-remove-unused-iscsi_queue_pdu-symbol-overl.patch
16lp-1894129-test-tool-BlockEraseReserved-override-driver-queue_p.patch
17lp-1894129-test-tool-Use-extern-int-in-headers.patch

Subscribers

People subscribed via source and target branches