gvirstor: in virstor_ctl_remove() the copy length of the second bcopy might be wrong.

Reviewed by: imp
Pull Request: https://github.com/freebsd/freebsd-src/pull/1763

This commit is contained in:

Wuyang Chung

2025-07-13 00:09:36 +08:00

committed by

Warner Losh

parent f388c311d3

commit f4e7cb535b

1 changed files with 1 additions and 1 deletions

									
										sys/geom/virstor/g_virstor.c
									
		+1
		-1
	
												View File
												
				@@ -589,7 +589,7 @@ virstor_ctl_remove(struct gctl_req *req, struct g_class *cp)

						    M_GVIRSTOR, M_WAITOK | M_ZERO);

						bcopy(sc->components, newcomp, found * sizeof(*sc->components));

						bcopy(&sc->components[found + 1], newcomp + found,

						    found * sizeof(*sc->components));

						    (sc->n_components - (found + 1)) * sizeof(*sc->components));

						if ((sc->components[j].flags & VIRSTOR_PROVIDER_ALLOCATED) != 0) {

							LOG_MSG(LVL_ERROR, "Allocated provider %s cannot be "

							    "removed from %s",