diff options
author | mjg <mjg@FreeBSD.org> | 2016-01-27 19:55:02 +0000 |
---|---|---|
committer | mjg <mjg@FreeBSD.org> | 2016-01-27 19:55:02 +0000 |
commit | 0e3463672209718efb883b2e8b3580c7ea014463 (patch) | |
tree | f519ea8bba560a6c7b17d92ed49628ab4f1f74c0 /sys/kern/kern_ktrace.c | |
parent | 4019d3df8f10fcc6dc8a7e78356272af60783b5a (diff) | |
download | FreeBSD-src-0e3463672209718efb883b2e8b3580c7ea014463.zip FreeBSD-src-0e3463672209718efb883b2e8b3580c7ea014463.tar.gz |
ktrace: tidy up ktrstruct
- minor style fixes
- avoid doing strlen twice [1]
PR: 206648
Submitted by: C Turt <ecturt gmail.com> (original version) [1]
Diffstat (limited to 'sys/kern/kern_ktrace.c')
-rw-r--r-- | sys/kern/kern_ktrace.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/sys/kern/kern_ktrace.c b/sys/kern/kern_ktrace.c index 367610b3..139fd31 100644 --- a/sys/kern/kern_ktrace.c +++ b/sys/kern/kern_ktrace.c @@ -757,15 +757,16 @@ ktrstruct(name, data, datalen) size_t datalen; { struct ktr_request *req; - char *buf = NULL; - size_t buflen; + char *buf; + size_t buflen, namelen; - if (!data) + if (data == NULL) datalen = 0; - buflen = strlen(name) + 1 + datalen; + namelen = strlen(name) + 1; + buflen = namelen + datalen; buf = malloc(buflen, M_KTRACE, M_WAITOK); strcpy(buf, name); - bcopy(data, buf + strlen(name) + 1, datalen); + bcopy(data, buf + namelen, datalen); if ((req = ktr_getrequest(KTR_STRUCT)) == NULL) { free(buf, M_KTRACE); return; |