summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorcandychencan <[email protected]>2016-09-21 02:27:15 +0800
committerBrian Behlendorf <[email protected]>2016-09-20 11:27:15 -0700
commitd2a6f65cfb65ec3079f843542a156704264260be (patch)
tree1d86e553f6984c3e2642dfc08ca35b4ac9c2540b
parent792517389fad5c495a2738b61c2e9c65dedaaa9a (diff)
Fix strncpy in taskq_create
Assign the copy length to TASKQ_NAMELEN, so if the name length equals 'TASKQ_NAMELEN+1' , the final '\0' of tq->tq_name is preserved. Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: candychencan <[email protected]> Closes #5136
-rw-r--r--lib/libzpool/taskq.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libzpool/taskq.c b/lib/libzpool/taskq.c
index e42e4261f..c1f87e173 100644
--- a/lib/libzpool/taskq.c
+++ b/lib/libzpool/taskq.c
@@ -268,7 +268,7 @@ taskq_create(const char *name, int nthreads, pri_t pri,
cv_init(&tq->tq_dispatch_cv, NULL, CV_DEFAULT, NULL);
cv_init(&tq->tq_wait_cv, NULL, CV_DEFAULT, NULL);
cv_init(&tq->tq_maxalloc_cv, NULL, CV_DEFAULT, NULL);
- (void) strncpy(tq->tq_name, name, TASKQ_NAMELEN + 1);
+ (void) strncpy(tq->tq_name, name, TASKQ_NAMELEN);
tq->tq_flags = flags | TASKQ_ACTIVE;
tq->tq_active = nthreads;
tq->tq_nthreads = nthreads;