svn commit: r297618 - head/sbin/fsck_msdosfs

Pedro F. Giffuni pfg at FreeBSD.org
Wed Apr 6 15:28:28 UTC 2016


Author: pfg
Date: Wed Apr  6 15:28:26 2016
New Revision: 297618
URL: https://svnweb.freebsd.org/changeset/base/297618

Log:
  fsck_msdosfs(8): Optimimize memsets
  
  Obtained from:	NetBSD (bin/50908)
  MFC after:	2 weeks

Modified:
  head/sbin/fsck_msdosfs/dir.c

Modified: head/sbin/fsck_msdosfs/dir.c
==============================================================================
--- head/sbin/fsck_msdosfs/dir.c	Wed Apr  6 14:16:37 2016	(r297617)
+++ head/sbin/fsck_msdosfs/dir.c	Wed Apr  6 15:28:26 2016	(r297618)
@@ -925,6 +925,7 @@ int
 reconnect(int dosfs, struct bootblock *boot, struct fatEntry *fat, cl_t head)
 {
 	struct dosDirEntry d;
+	int len;
 	u_char *p;
 
 	if (!ask(1, "Reconnect"))
@@ -976,14 +977,15 @@ reconnect(int dosfs, struct bootblock *b
 	boot->NumFiles++;
 	/* Ensure uniqueness of entry here!				XXX */
 	memset(&d, 0, sizeof d);
-	(void)snprintf(d.name, sizeof(d.name), "%u", head);
+	/* worst case -1 = 4294967295, 10 digits */
+	len = snprintf(d.name, sizeof(d.name), "%u", head);
 	d.flags = 0;
 	d.head = head;
 	d.size = fat[head].length * boot->ClusterSize;
 
-	memset(p, 0, 32);
-	memset(p, ' ', 11);
-	memcpy(p, d.name, strlen(d.name));
+	memcpy(p, d.name, len);
+	memset(p + len, ' ', 11 - len);
+	memset(p + 11, 0, 32 - 11);
 	p[26] = (u_char)d.head;
 	p[27] = (u_char)(d.head >> 8);
 	if (boot->ClustMask == CLUST32_MASK) {


More information about the svn-src-head mailing list