Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 3 Jan 2012 19:10:12 GMT
From:      dfilter@FreeBSD.ORG (dfilter service)
To:        freebsd-fs@FreeBSD.org
Subject:   Re: kern/142401: commit references a PR
Message-ID:  <201201031910.q03JAClo012365@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help
The following reply was made to PR kern/142401; it has been noted by GNATS.

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/142401: commit references a PR
Date: Tue,  3 Jan 2012 19:09:11 +0000 (UTC)

 Author: pfg
 Date: Tue Jan  3 19:09:01 2012
 New Revision: 229407
 URL: http://svn.freebsd.org/changeset/base/229407
 
 Log:
   Minor cleanups to ntfs code
   
   bzero -> memset
   rename variables to avoid shadowing.
   
   PR:		142401
   Obtained from:	NetBSD
   Approved by	jhb (mentor)
 
 Modified:
   head/sys/fs/ntfs/ntfs_compr.c
   head/sys/fs/ntfs/ntfs_subr.c
 
 Modified: head/sys/fs/ntfs/ntfs_compr.c
 ==============================================================================
 --- head/sys/fs/ntfs/ntfs_compr.c	Tue Jan  3 19:01:54 2012	(r229406)
 +++ head/sys/fs/ntfs/ntfs_compr.c	Tue Jan  3 19:09:01 2012	(r229407)
 @@ -42,7 +42,7 @@
  
  int
  ntfs_uncompblock(
 -	u_int8_t * buf,
 +	u_int8_t * dbuf,
  	u_int8_t * cbuf)
  {
  	u_int32_t       ctag;
 @@ -60,8 +60,8 @@ ntfs_uncompblock(
  			dprintf(("ntfs_uncompblock: len: %x instead of %d\n",
  			    len, 0xfff));
  		}
 -		memcpy(buf, cbuf + 2, len + 1);
 -		bzero(buf + len + 1, NTFS_COMPBLOCK_SIZE - 1 - len);
 +		memcpy(dbuf, cbuf + 2, len + 1);
 +		memset(dbuf + len + 1, 0, NTFS_COMPBLOCK_SIZE - 1 - len);
  		return len + 3;
  	}
  	cpos = 2;
 @@ -78,12 +78,12 @@ ntfs_uncompblock(
  				boff = -1 - (GET_UINT16(cbuf + cpos) >> dshift);
  				blen = 3 + (GET_UINT16(cbuf + cpos) & lmask);
  				for (j = 0; (j < blen) && (pos < NTFS_COMPBLOCK_SIZE); j++) {
 -					buf[pos] = buf[pos + boff];
 +					dbuf[pos] = dbuf[pos + boff];
  					pos++;
  				}
  				cpos += 2;
  			} else {
 -				buf[pos++] = cbuf[cpos++];
 +				dbuf[pos++] = cbuf[cpos++];
  			}
  			ctag >>= 1;
  		}
 
 Modified: head/sys/fs/ntfs/ntfs_subr.c
 ==============================================================================
 --- head/sys/fs/ntfs/ntfs_subr.c	Tue Jan  3 19:01:54 2012	(r229406)
 +++ head/sys/fs/ntfs/ntfs_subr.c	Tue Jan  3 19:09:01 2012	(r229407)
 @@ -1636,7 +1636,7 @@ ntfs_readntvattr_plain(
  					for(; remains; remains--)
  						uiomove("", 1, uio);
  				} else 
 -					bzero(data, tocopy);
 +					memset(data, 0, tocopy);
  				data = data + tocopy;
  			}
  			cnt++;
 @@ -1783,7 +1783,7 @@ ntfs_readattr(
  						uiomove("", 1, uio);
  				}
  				else
 -					bzero(data, tocopy);
 +					memset(data, 0, tocopy);
  			} else {
  				error = ntfs_uncompunit(ntmp, uup, cup);
  				if (error)
 _______________________________________________
 svn-src-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-src-all
 To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201201031910.q03JAClo012365>