From efbe78447829bf5616d630f283e6ddbde617dc26 Mon Sep 17 00:00:00 2001 From: CentOS Sources Date: Tue, 28 Mar 2023 09:24:24 +0000 Subject: [PATCH] import rdma-core-44.0-2.el9 --- .gitignore | 2 +- .rdma-core.metadata | 2 +- ...util-fix-overflow-in-remap_node_name.patch | 85 +++++++++++++++++ ...-drop-unnecessary-nodedesc-local-cop.patch | 95 +++++++++++++++++++ ...le-providers-that-were-not-enabled-i.patch | 53 +++++++++++ ...-printing-a-possibly-non-NUL-termina.patch | 38 ++++++++ ...t-perform-device-rename-on-OPA-devi.patch} | 0 ...RNEL-as-default-interface-naming-co.patch} | 0 SPECS/rdma-core.spec | 46 ++++----- 9 files changed, 297 insertions(+), 24 deletions(-) create mode 100644 SOURCES/0001-util-fix-overflow-in-remap_node_name.patch create mode 100644 SOURCES/0002-infiniband-diags-drop-unnecessary-nodedesc-local-cop.patch create mode 100644 SOURCES/0003-CMakeLists-disable-providers-that-were-not-enabled-i.patch create mode 100644 SOURCES/0003-libibnetdisc-fix-printing-a-possibly-non-NUL-termina.patch rename SOURCES/{0001-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch => 9998-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch} (100%) rename SOURCES/{udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch => 9999-udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch} (100%) diff --git a/.gitignore b/.gitignore index 8a83a67..a007d05 100644 --- a/.gitignore +++ b/.gitignore @@ -1 +1 @@ -SOURCES/rdma-core-41.0.tar.gz +SOURCES/rdma-core-44.0.tar.gz diff --git a/.rdma-core.metadata b/.rdma-core.metadata index 4c63b9d..43c389f 100644 --- a/.rdma-core.metadata +++ b/.rdma-core.metadata @@ -1 +1 @@ -6ed8cf4056ed8250cf94fa14e79d76fb74ad4548 SOURCES/rdma-core-41.0.tar.gz +f1ff3710270ad7920626f73e955aae2b1a043136 SOURCES/rdma-core-44.0.tar.gz diff --git a/SOURCES/0001-util-fix-overflow-in-remap_node_name.patch b/SOURCES/0001-util-fix-overflow-in-remap_node_name.patch new file mode 100644 index 0000000..860c153 --- /dev/null +++ b/SOURCES/0001-util-fix-overflow-in-remap_node_name.patch @@ -0,0 +1,85 @@ +From 5075b961a29ff9c418e1fefe78432e95dd0a5fcc Mon Sep 17 00:00:00 2001 +From: Michal Schmidt +Date: Wed, 1 Feb 2023 22:41:06 +0100 +Subject: [PATCH 1/3] util: fix overflow in remap_node_name() + +The function remap_node_name() assumes the parameter 'nodedesc' is at +least IB_SMP_DATA_SIZE + 1 (i.e. 65) bytes long, because it passes it to +clean_nodedesc() that writes a nul-terminator to it at offset +IB_SMP_DATA_SIZE. Callers in infiniband-diags/saquery.c pass +a (struct ib_node_desc_t).description as the argument, which is only +IB_NODE_DESCRIPTION_SIZE (i.e. 64) bytes long. This is an overflow. + +An odd thing about remap_node_name() is that it may (but does not +always) rewrite the nodedesc in-place. Callers do not appear to +appreciate this behavior. Most of them are various print_* and dump_* +functions where rewriting the input makes no sense. Some callers make a +local copy of the nodedesc first, possibly to protect the original. +One caller (infiniband-diags/saquery.c:print_node_records()) checks if +either the original description or the remapped one matches a given +requested_name - so it looks like it prefers the original to be +not rewritten. + +Let's make remap_node_name() a bit safer and more convenient to use. +Allocate a fixed-sized copy first. Then use strncpy to copy from +'nodedesc', never reading more than IB_SMP_DATA_SIZE (64) bytes. +Apply clean_nodedesc() on the correctly-sized copy. This solves the +overflow bug. Also, the in-place rewrite of 'nodedesc' is gone and it +can become a (const char*). + +The overflow was found by a static checker (covscan). + +Fixes: d974c4e398d2 ("Fix max length of node description (ibnetdiscover and smpquery)") +Signed-off-by: Michal Schmidt +--- + util/node_name_map.c | 12 +++++++++--- + util/node_name_map.h | 3 +-- + 2 files changed, 10 insertions(+), 5 deletions(-) + +diff --git a/util/node_name_map.c b/util/node_name_map.c +index 30b73eb1448e..511cb92ef19c 100644 +--- a/util/node_name_map.c ++++ b/util/node_name_map.c +@@ -95,7 +95,7 @@ void close_node_name_map(nn_map_t * map) + free(map); + } + +-char *remap_node_name(nn_map_t * map, uint64_t target_guid, char *nodedesc) ++char *remap_node_name(nn_map_t * map, uint64_t target_guid, const char *nodedesc) + { + char *rc = NULL; + name_map_item_t *item = NULL; +@@ -108,8 +108,14 @@ char *remap_node_name(nn_map_t * map, uint64_t target_guid, char *nodedesc) + rc = strdup(item->name); + + done: +- if (rc == NULL) +- rc = strdup(clean_nodedesc(nodedesc)); ++ if (rc == NULL) { ++ rc = malloc(IB_SMP_DATA_SIZE + 1); ++ if (rc) { ++ strncpy(rc, nodedesc, IB_SMP_DATA_SIZE); ++ rc[IB_SMP_DATA_SIZE] = '\0'; ++ clean_nodedesc(rc); ++ } ++ } + return (rc); + } + +diff --git a/util/node_name_map.h b/util/node_name_map.h +index e78d274b116e..d83d672782c4 100644 +--- a/util/node_name_map.h ++++ b/util/node_name_map.h +@@ -12,8 +12,7 @@ typedef struct nn_map nn_map_t; + + nn_map_t *open_node_name_map(const char *node_name_map); + void close_node_name_map(nn_map_t *map); +-/* NOTE: parameter "nodedesc" may be modified here. */ +-char *remap_node_name(nn_map_t *map, uint64_t target_guid, char *nodedesc); ++char *remap_node_name(nn_map_t *map, uint64_t target_guid, const char *nodedesc); + char *clean_nodedesc(char *nodedesc); + + #endif +-- +2.39.1 + diff --git a/SOURCES/0002-infiniband-diags-drop-unnecessary-nodedesc-local-cop.patch b/SOURCES/0002-infiniband-diags-drop-unnecessary-nodedesc-local-cop.patch new file mode 100644 index 0000000..7927ba6 --- /dev/null +++ b/SOURCES/0002-infiniband-diags-drop-unnecessary-nodedesc-local-cop.patch @@ -0,0 +1,95 @@ +From d5723a0f69577fd3022024ca17c27e273a29695b Mon Sep 17 00:00:00 2001 +From: Michal Schmidt +Date: Wed, 1 Feb 2023 22:41:16 +0100 +Subject: [PATCH 2/3] infiniband-diags: drop unnecessary nodedesc local copies + +Now that remap_node_name() never rewrites nodedesc in-place, some +copying can be avoided. + +Signed-off-by: Michal Schmidt +--- + infiniband-diags/dump_fts.c | 14 +++----------- + 1 file changed, 3 insertions(+), 11 deletions(-) + +diff --git a/infiniband-diags/dump_fts.c b/infiniband-diags/dump_fts.c +index ce6bfb9ecc33..acef9efe692d 100644 +--- a/infiniband-diags/dump_fts.c ++++ b/infiniband-diags/dump_fts.c +@@ -109,7 +109,6 @@ static void dump_multicast_tables(ibnd_node_t *node, unsigned startl, + unsigned endl, struct ibmad_port *mad_port) + { + ib_portid_t *portid = &node->path_portid; +- char nd[IB_SMP_DATA_SIZE + 1] = { 0 }; + char str[512]; + char *s; + uint64_t nodeguid; +@@ -119,7 +118,6 @@ static void dump_multicast_tables(ibnd_node_t *node, unsigned startl, + char *mapnd = NULL; + int n = 0; + +- memcpy(nd, node->nodedesc, strlen(node->nodedesc)); + nports = node->numports; + nodeguid = node->guid; + +@@ -149,7 +147,7 @@ static void dump_multicast_tables(ibnd_node_t *node, unsigned startl, + endl = IB_MAX_MCAST_LID; + } + +- mapnd = remap_node_name(node_name_map, nodeguid, nd); ++ mapnd = remap_node_name(node_name_map, nodeguid, node->nodedesc); + + printf("Multicast mlids [0x%x-0x%x] of switch %s guid 0x%016" PRIx64 + " (%s):\n", startl, endl, portid2str(portid), nodeguid, +@@ -224,8 +222,6 @@ static int dump_lid(char *str, int str_len, int lid, int valid, + ibnd_fabric_t *fabric, int *last_port_lid, + int *base_port_lid, uint64_t *portguid) + { +- char nd[IB_SMP_DATA_SIZE + 1] = { 0 }; +- + ibnd_port_t *port = NULL; + + char ntype[50], sguid[30]; +@@ -276,14 +272,12 @@ static int dump_lid(char *str, int str_len, int lid, int valid, + baselid = port->base_lid; + lmc = port->lmc; + +- memcpy(nd, port->node->nodedesc, strlen(port->node->nodedesc)); +- + if (lmc > 0) { + *base_port_lid = baselid; + *last_port_lid = baselid + (1 << lmc) - 1; + } + +- mapnd = remap_node_name(node_name_map, nodeguid, nd); ++ mapnd = remap_node_name(node_name_map, nodeguid, port->node->nodedesc); + + rc = snprintf(str, str_len, ": (%s portguid %s: '%s')", + mad_dump_val(IB_NODE_TYPE_F, ntype, sizeof ntype, +@@ -302,7 +296,6 @@ static void dump_unicast_tables(ibnd_node_t *node, int startl, int endl, + { + ib_portid_t * portid = &node->path_portid; + char lft[IB_SMP_DATA_SIZE] = { 0 }; +- char nd[IB_SMP_DATA_SIZE + 1] = { 0 }; + char str[200]; + uint64_t nodeguid; + int block, i, e, top; +@@ -315,7 +308,6 @@ static void dump_unicast_tables(ibnd_node_t *node, int startl, int endl, + mad_decode_field(node->switchinfo, IB_SW_LINEAR_FDB_TOP_F, &top); + nodeguid = node->guid; + nports = node->numports; +- memcpy(nd, node->nodedesc, strlen(node->nodedesc)); + + if (!endl || endl > top) + endl = top; +@@ -326,7 +318,7 @@ static void dump_unicast_tables(ibnd_node_t *node, int startl, int endl, + endl = IB_MAX_UCAST_LID; + } + +- mapnd = remap_node_name(node_name_map, nodeguid, nd); ++ mapnd = remap_node_name(node_name_map, nodeguid, node->nodedesc); + + printf("Unicast lids [0x%x-0x%x] of switch %s guid 0x%016" PRIx64 + " (%s):\n", startl, endl, portid2str(portid), nodeguid, +-- +2.39.1 + diff --git a/SOURCES/0003-CMakeLists-disable-providers-that-were-not-enabled-i.patch b/SOURCES/0003-CMakeLists-disable-providers-that-were-not-enabled-i.patch new file mode 100644 index 0000000..84f1b16 --- /dev/null +++ b/SOURCES/0003-CMakeLists-disable-providers-that-were-not-enabled-i.patch @@ -0,0 +1,53 @@ +From eff6b07e92a1674818c5d8c9993651dbbeabccf4 Mon Sep 17 00:00:00 2001 +From: Michal Schmidt +Date: Wed, 1 Feb 2023 15:24:23 +0100 +Subject: [PATCH 3/5] CMakeLists: disable providers that were not enabled in + RHEL 9.1 + +Doing a package rebase, but don't want to enable additional drivers +unless explicitly requested. + +Upstream Status: RHEL only + +Signed-off-by: Michal Schmidt +--- + CMakeLists.txt | 12 ++++++------ + 1 file changed, 6 insertions(+), 6 deletions(-) + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index bac10516bb85..b7eca65f0fe2 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -711,23 +711,23 @@ add_subdirectory(providers/bnxt_re) + add_subdirectory(providers/cxgb4) # NO SPARSE + add_subdirectory(providers/efa) + add_subdirectory(providers/efa/man) +-add_subdirectory(providers/erdma) ++#add_subdirectory(providers/erdma) + add_subdirectory(providers/hns) + add_subdirectory(providers/irdma) +-add_subdirectory(providers/mana) +-add_subdirectory(providers/mana/man) ++#add_subdirectory(providers/mana) ++#add_subdirectory(providers/mana/man) + add_subdirectory(providers/mlx4) + add_subdirectory(providers/mlx4/man) + add_subdirectory(providers/mlx5) + add_subdirectory(providers/mlx5/man) +-add_subdirectory(providers/mthca) +-add_subdirectory(providers/ocrdma) ++#add_subdirectory(providers/mthca) ++#add_subdirectory(providers/ocrdma) + add_subdirectory(providers/qedr) + add_subdirectory(providers/vmw_pvrdma) + endif() + + add_subdirectory(providers/hfi1verbs) +-add_subdirectory(providers/ipathverbs) ++#add_subdirectory(providers/ipathverbs) + add_subdirectory(providers/rxe) + add_subdirectory(providers/rxe/man) + add_subdirectory(providers/siw) +-- +2.39.1 + diff --git a/SOURCES/0003-libibnetdisc-fix-printing-a-possibly-non-NUL-termina.patch b/SOURCES/0003-libibnetdisc-fix-printing-a-possibly-non-NUL-termina.patch new file mode 100644 index 0000000..2a5cc84 --- /dev/null +++ b/SOURCES/0003-libibnetdisc-fix-printing-a-possibly-non-NUL-termina.patch @@ -0,0 +1,38 @@ +From 45fcc7ad41216a93bafb452f7d7a4507d52722cd Mon Sep 17 00:00:00 2001 +From: Michal Schmidt +Date: Wed, 1 Feb 2023 23:30:52 +0100 +Subject: [PATCH 3/3] libibnetdisc: fix printing a possibly non-NUL-terminated + string + +Found by a static check (covscan). + +Fixes: d974c4e398d2 ("Fix max length of node description (ibnetdiscover and smpquery)") +Signed-off-by: Michal Schmidt +--- + libibnetdisc/chassis.c | 3 ++- + 1 file changed, 2 insertions(+), 1 deletion(-) + +diff --git a/libibnetdisc/chassis.c b/libibnetdisc/chassis.c +index a3ec1d82807c..bc1a8aff8acb 100644 +--- a/libibnetdisc/chassis.c ++++ b/libibnetdisc/chassis.c +@@ -597,7 +597,7 @@ static int fill_mellanox_chassis_record(ibnd_node_t * node) + int p = 0; + ibnd_port_t *port; + +- char node_desc[IB_SMP_DATA_SIZE]; ++ char node_desc[IB_SMP_DATA_SIZE + 1]; + char *system_name; + char *system_type; + char *system_slot_name; +@@ -617,6 +617,7 @@ static int fill_mellanox_chassis_record(ibnd_node_t * node) + */ + + memcpy(node_desc, node->nodedesc, IB_SMP_DATA_SIZE); ++ node_desc[IB_SMP_DATA_SIZE] = '\0'; + + IBND_DEBUG("fill_mellanox_chassis_record: node_desc:%s \n",node_desc); + +-- +2.39.1 + diff --git a/SOURCES/0001-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch b/SOURCES/9998-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch similarity index 100% rename from SOURCES/0001-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch rename to SOURCES/9998-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch diff --git a/SOURCES/udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch b/SOURCES/9999-udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch similarity index 100% rename from SOURCES/udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch rename to SOURCES/9999-udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch diff --git a/SPECS/rdma-core.spec b/SPECS/rdma-core.spec index b97b6a2..7b10299 100644 --- a/SPECS/rdma-core.spec +++ b/SPECS/rdma-core.spec @@ -1,6 +1,6 @@ Name: rdma-core -Version: 41.0 -Release: 3%{?dist} +Version: 44.0 +Release: 2%{?dist} Summary: RDMA core userspace libraries and daemons # Almost everything is licensed under the OFA dual GPLv2, 2 Clause BSD license @@ -10,8 +10,13 @@ Summary: RDMA core userspace libraries and daemons License: GPLv2 or BSD Url: https://github.com/linux-rdma/rdma-core Source: https://github.com/linux-rdma/rdma-core/releases/download/v%{version}/%{name}-%{version}.tar.gz -Patch1: 0001-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch -Patch2: udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch +# 0001-0003: https://github.com/linux-rdma/rdma-core/pull/1308 +Patch1: 0001-util-fix-overflow-in-remap_node_name.patch +Patch2: 0002-infiniband-diags-drop-unnecessary-nodedesc-local-cop.patch +Patch3: 0003-libibnetdisc-fix-printing-a-possibly-non-NUL-termina.patch +Patch9000: 0003-CMakeLists-disable-providers-that-were-not-enabled-i.patch +Patch9998: 9998-kernel-boot-Do-not-perform-device-rename-on-OPA-devi.patch +Patch9999: 9999-udev-keep-NAME_KERNEL-as-default-interface-naming-co.patch # Do not build static libs by default. %define with_static %{?_with_static: 1} %{?!_with_static: 0} @@ -263,11 +268,15 @@ easy, object-oriented access to IB verbs. %prep %setup -q -%if 0%{?fedora} %patch1 -p1 +%patch2 -p1 +%patch3 -p1 +%if 0%{?fedora} +%patch9998 -p1 %endif %if 0%{?rhel} -%patch2 -p1 +%patch9000 -p1 +%patch9999 -p1 %endif %build @@ -323,7 +332,6 @@ mkdir -p %{buildroot}/%{_sysconfdir}/rdma # Red Hat specific glue %global dracutlibdir %{_prefix}/lib/dracut %global sysmodprobedir %{_prefix}/lib/modprobe.d -mkdir -p %{buildroot}%{_sysconfdir}/udev/rules.d mkdir -p %{buildroot}%{_libexecdir} mkdir -p %{buildroot}%{_udevrulesdir} mkdir -p %{buildroot}%{dracutlibdir}/modules.d/05rdma @@ -344,18 +352,6 @@ install -D -m0644 redhat/rdma.conf %{buildroot}%{_sysconfdir}/rdma/modules/rdma. rm -rf %{buildroot}/%{_initrddir}/ rm -f %{buildroot}/%{_sbindir}/srp_daemon.sh -# Remove ibverbs provider libs we don't support -rm -f %{buildroot}/%{_libdir}/libibverbs/libcxgb3-rdmav*.so -rm -f %{buildroot}/%{_sysconfdir}/libibverbs.d/cxgb3.driver -rm -f %{buildroot}/%{_libdir}/libibverbs/libocrdma-rdmav*.so -rm -f %{buildroot}/%{_sysconfdir}/libibverbs.d/ocrdma.driver -rm -f %{buildroot}/%{_libdir}/libibverbs/libnes-rdmav*.so -rm -f %{buildroot}/%{_sysconfdir}/libibverbs.d/nes.driver -rm -f %{buildroot}/%{_libdir}/libibverbs/libmthca-rdmav*.so -rm -f %{buildroot}/%{_sysconfdir}/libibverbs.d/mthca.driver -rm -f %{buildroot}/%{_libdir}/libibverbs/libipathverbs-rdmav*.so -rm -f %{buildroot}/%{_sysconfdir}/libibverbs.d/ipathverbs.driver - %ldconfig_scriptlets -n libibverbs %ldconfig_scriptlets -n libibumad @@ -393,6 +389,7 @@ fi %files %dir %{_sysconfdir}/rdma %dir %{_docdir}/%{name} +%doc %{_docdir}/%{name}/70-persistent-ipoib.rules %doc %{_docdir}/%{name}/README.md %doc %{_docdir}/%{name}/rxe.md %doc %{_docdir}/%{name}/udev.md @@ -403,10 +400,8 @@ fi %config(noreplace) %{_sysconfdir}/rdma/modules/opa.conf %config(noreplace) %{_sysconfdir}/rdma/modules/rdma.conf %config(noreplace) %{_sysconfdir}/rdma/modules/roce.conf -%config(noreplace) %{_sysconfdir}/udev/rules.d/* %dir %{_sysconfdir}/modprobe.d %config(noreplace) %{_sysconfdir}/modprobe.d/mlx4.conf -%config(noreplace) %{_sysconfdir}/modprobe.d/truescale.conf %{_unitdir}/rdma-hw.target %{_unitdir}/rdma-load-modules@.service %dir %{dracutlibdir} @@ -424,7 +419,6 @@ fi %dir %{sysmodprobedir} %{sysmodprobedir}/libmlx4.conf %{_libexecdir}/mlx4-setup.sh -%{_libexecdir}/truescale-serdes.cmds %{_sbindir}/rdma-ndd %{_unitdir}/rdma-ndd.service %{_mandir}/man7/rxe* @@ -622,6 +616,14 @@ fi %endif %changelog +* Wed Feb 01 2023 Michal Schmidt - 44.0-2 +- Fix covscan-found issues. +- Resolves: rhbz#2112984, rhbz#2142687 + +* Wed Feb 01 2023 Michal Schmidt - 44.0-1 +- Rebase to upstream release v44.0 +- Resolves: rhbz#2112984, rhbz#2142687 + * Tue Aug 02 2022 Michal Schmidt - 41.0-3 - Rebase to upstream release v41.0 - Resolves: rhbz#2049521