Page MenuHomeFreeBSD
Feed Advanced Search

Mar 13 2017

pfg committed rS315213: nscd(8): let calloc(3) do the multiplying..
nscd(8): let calloc(3) do the multiplying.
Mar 13 2017, 8:35 PM
pfg committed rS315212: mkimg(1): let calloc(3) do the multiplication..
mkimg(1): let calloc(3) do the multiplication.
Mar 13 2017, 8:14 PM
pfg committed rS315187: Minor libc cleanup: let calloc(3) do the multiplication..
Minor libc cleanup: let calloc(3) do the multiplication.
Mar 13 2017, 4:14 AM

Mar 12 2017

pfg resigned from D9982: Fix a memory leak in lpr, alternate checks....

Phabricator has been hitting false build positives, don't worry.
I'll stand down as reviewer... I have too much to do on other fronts :(.

Mar 12 2017, 10:33 PM
pfg committed rS315164: Revert r314780.
Revert r314780
Mar 12 2017, 5:42 PM
pfg updated the diff for D9980: lib: make use of reallocarray(3)..

Minor enhancement in libthread_db, move a multiplier to the second
argument to reduce risk of an overflow in the first argument.

Mar 12 2017, 4:21 PM
pfg created D9980: lib: make use of reallocarray(3)..
Mar 12 2017, 4:16 PM
pfg committed rS315162: libc: provide some bounds-checking through reallocarray(3)..
libc: provide some bounds-checking through reallocarray(3).
Mar 12 2017, 4:03 PM
pfg closed D9955: Make use of reallocarray(3) in libc. by committing rS315162: libc: provide some bounds-checking through reallocarray(3)..
Mar 12 2017, 4:03 PM
pfg committed rS315135: MFC r314321:.
MFC r314321:
Mar 12 2017, 5:36 AM
pfg committed rS315134: MFC r314321:.
MFC r314321:
Mar 12 2017, 5:35 AM
pfg updated the diff for D9955: Make use of reallocarray(3) in libc..

Forced update to restart the build.

Mar 12 2017, 4:01 AM
pfg committed rS315104: libc: small cleanup..
libc: small cleanup.
Mar 12 2017, 3:50 AM
pfg committed rS315097: libc: small cleanups..
libc: small cleanups.
Mar 12 2017, 3:29 AM
pfg committed rS315096: libc: small cleanups..
libc: small cleanups.
Mar 12 2017, 3:26 AM
pfg committed rS315095: libc: small cleanups..
libc: small cleanups.
Mar 12 2017, 3:22 AM
pfg updated the diff for D9955: Make use of reallocarray(3) in libc..

Update: just for reference ... I will be committing some
cleanups independently as suggested by ngie.

Mar 12 2017, 2:59 AM
pfg added inline comments to D9955: Make use of reallocarray(3) in libc..
Mar 12 2017, 2:33 AM
pfg added a comment to D9955: Make use of reallocarray(3) in libc..

Some notes here b/c Phabricator doesn't let me reply:

Mar 12 2017, 2:21 AM
pfg added a comment to D9954: Fix a memory leak in lpr.
In D9954#205869, @ngie wrote:

That's not the only fun... I'd probably incorporate this patch as well, but commit the calloc change separately:

$ svn diff usr.sbin/lpr/chkprintcap/
Index: usr.sbin/lpr/chkprintcap/chkprintcap.c
===================================================================
--- usr.sbin/lpr/chkprintcap/skimprintcap.c    (revision 314942)
+++ usr.sbin/lpr/chkprintcap/skimprintcap.c    (working copy)
@@ -81,8 +81,9 @@
        enum {NO_CONTINUE, WILL_CONTINUE, BAD_CONTINUE} is_cont, had_cont;
        enum {CMNT_LINE, ENTRY_LINE, TAB_LINE, TABERR_LINE} is_type, had_type;

-       skinf = malloc(sizeof(struct skiminfo));
-       memset(skinf, 0, sizeof(struct skiminfo));
+       skinf = calloc(1, sizeof(struct skiminfo));
+       if (skinf == NULL)
+               return (NULL);

        pc_file = fopen(pcap_fname, "r");
        if (pc_file == NULL) {
Mar 12 2017, 2:07 AM

Mar 11 2017

pfg created D9955: Make use of reallocarray(3) in libc..
Mar 11 2017, 4:21 AM
pfg added a reviewer for D9954: Fix a memory leak in lpr: gad.

Add gad@ as he is the maintainer.
For the record: Coverity doesn't report any of these.

Mar 11 2017, 4:03 AM

Mar 10 2017

pfg abandoned D9915: Use of reallocarray(3) in userland..

Please do not commit as-is.

Mar 10 2017, 7:14 PM
pfg committed rS315007: localedef(1): Add comment markings for license..
localedef(1): Add comment markings for license.
Mar 10 2017, 4:12 PM
pfg committed rS315006: localedef(1): Fix mismatch..
localedef(1): Fix mismatch.
Mar 10 2017, 4:06 PM

Mar 9 2017

pfg committed rS314974: localedef(1): Fix small coverity issues..
localedef(1): Fix small coverity issues.
Mar 9 2017, 9:49 PM
pfg committed rS314972: Revert r314969, r314961:.
Revert r314969, r314961:
Mar 9 2017, 7:02 PM
pfg committed rS314969: localedef(1): Fix mismatch in previous commit..
localedef(1): Fix mismatch in previous commit.
Mar 9 2017, 6:07 PM
pfg committed rS314961: localedef(1): Fix for memory leaks reported by coverity..
localedef(1): Fix for memory leaks reported by coverity.
Mar 9 2017, 3:21 PM
pfg committed rS314939: MFC r314145, r314158.
MFC r314145, r314158
Mar 9 2017, 2:59 AM
pfg committed rS314938: MFC r314145, r314158.
MFC r314145, r314158
Mar 9 2017, 2:58 AM
pfg added a reverting change for rS294545: Revert r294271:: rS314937: Revert 294545:.
Mar 9 2017, 2:47 AM
pfg committed rS314937: Revert 294545:.
Revert 294545:
Mar 9 2017, 2:47 AM

Mar 8 2017

pfg committed rS314922: MFC r314505:.
MFC r314505:
Mar 8 2017, 10:40 PM
pfg committed rS314921: MFC r314505:.
MFC r314505:
Mar 8 2017, 10:39 PM
pfg resigned from D3043: _FORTIFY_SOURCE support.
In D3043#205126, @ngie wrote:

Is this review still valid?

Mar 8 2017, 5:55 PM ยท fortify source

Mar 7 2017

pfg committed rS314877: lpr(1): small bounds check with reallocarray(3)..
lpr(1): small bounds check with reallocarray(3).
Mar 7 2017, 7:30 PM
pfg retitled D9915: Use of reallocarray(3) in userland. from to Extensive use of reallocarray(3) in userland..
Mar 7 2017, 1:37 AM

Mar 6 2017

pfg committed rS314780: libpam: extra bounds checking through reallocarray(3)..
libpam: extra bounds checking through reallocarray(3).
Mar 6 2017, 3:45 PM
pfg committed rS314779: Revert r314777: wrong log, the change was to libpam..
Revert r314777: wrong log, the change was to libpam.
Mar 6 2017, 3:42 PM
pfg committed rS314778: libfetch: extra bounds checking through reallocarray(3)..
libfetch: extra bounds checking through reallocarray(3).
Mar 6 2017, 3:38 PM
pfg committed rS314777: libfetch: extra bounds checking through reallocarray(3)..
libfetch: extra bounds checking through reallocarray(3).
Mar 6 2017, 3:37 PM
pfg committed rS314721: Revert r314669, r314670:.
Revert r314669, r314670:
Mar 6 2017, 3:52 AM

Mar 5 2017

pfg committed rS314704: bc/dc/patch: make some use of reallocarray(3)..
bc/dc/patch: make some use of reallocarray(3).
Mar 5 2017, 4:10 PM

Mar 4 2017

pfg committed rS314670: Add UPDATING entry per r314669: removal of classic i486 configuration..
Add UPDATING entry per r314669: removal of classic i486 configuration.
Mar 4 2017, 3:23 PM
pfg closed D9879: Drop i486 from the default GENERIC i386 kernel..

Committed as r314669 but I missed the "Differential Revision" reference.

Mar 4 2017, 3:08 PM
pfg committed rS314669: Drop i486 from the default i386 GENERIC kernel configuration..
Drop i486 from the default i386 GENERIC kernel configuration.
Mar 4 2017, 3:04 PM
pfg added a comment to D9879: Drop i486 from the default GENERIC i386 kernel..
In D9879#204076, @imp wrote:

This is fine, though the commit message could use some work.

Why not remove I586_CPU also?

Mar 4 2017, 2:55 PM

Mar 3 2017

pfg added a comment to D9879: Drop i486 from the default GENERIC i386 kernel..
In D9879#204076, @imp wrote:

This is fine, though the commit message could use some work.

Why not remove I586_CPU also?

Mar 3 2017, 11:55 PM
pfg updated D9879: Drop i486 from the default GENERIC i386 kernel..
Mar 3 2017, 9:53 PM
pfg retitled D9879: Drop i486 from the default GENERIC i386 kernel. from to Drop i486 from the default GENERIC i386 kernel..
Mar 3 2017, 9:41 PM

Mar 1 2017

pfg committed rS314505: Split the ficl CFLAGS when they refer to an arch-specific include path..
Split the ficl CFLAGS when they refer to an arch-specific include path.
Mar 1 2017, 7:36 PM
pfg committed rS314445: MFC r314316:.
MFC r314316:
Mar 1 2017, 1:19 AM
pfg committed rS314444: MFC r314316:.
MFC r314316:
Mar 1 2017, 1:18 AM

Feb 28 2017

pfg committed rS314409: dc(1): Introduce e command, equivalent to p, but writes to stderr.
dc(1): Introduce e command, equivalent to p, but writes to stderr
Feb 28 2017, 8:51 PM

Feb 27 2017

pfg committed rS314322: librss: simplify some NULL checks..
librss: simplify some NULL checks.
Feb 27 2017, 12:10 AM

Feb 26 2017

pfg committed rS314321: dc(1): Merge minor changes from OpenBSD..
dc(1): Merge minor changes from OpenBSD.
Feb 26 2017, 10:17 PM
pfg committed rS314316: dc(1): Catch up with OpenBSD tag..
dc(1): Catch up with OpenBSD tag.
Feb 26 2017, 9:24 PM

Feb 24 2017

pfg committed rS314227: MFC r313897:.
MFC r313897:
Feb 24 2017, 9:38 PM
pfg committed rS314225: MFC r313897:.
MFC r313897:
Feb 24 2017, 9:36 PM
pfg committed rS314224: MFC r313554:.
MFC r313554:
Feb 24 2017, 9:22 PM
pfg committed rS314223: MFC r313554:.
MFC r313554:
Feb 24 2017, 9:21 PM

Feb 23 2017

pfg committed rS314186: at91: double assignment..
at91: double assignment.
Feb 23 2017, 11:49 PM
pfg committed rS314158: vxge(4): Yet another (minor) mismatch..
vxge(4): Yet another (minor) mismatch.
Feb 23 2017, 6:56 PM
pfg committed rS314156: rt_attach(): double assignment..
rt_attach(): double assignment.
Feb 23 2017, 6:15 PM
pfg committed rS314147: bktr(4): double assignment..
bktr(4): double assignment.
Feb 23 2017, 4:19 PM
pfg committed rS314145: dev/vxe: double assignments..
dev/vxe: double assignments.
Feb 23 2017, 4:10 PM
pfg committed rS314142: dev/ce: double assignment..
dev/ce: double assignment.
Feb 23 2017, 3:30 PM
pfg committed rS314141: rtadvd(8): double assignment..
rtadvd(8): double assignment.
Feb 23 2017, 3:11 PM
pfg committed rS314140: camdd(8): double assignment..
camdd(8): double assignment.
Feb 23 2017, 3:05 PM

Feb 22 2017

pfg committed rS314068: sys: Replace zero with NULL for pointers..
sys: Replace zero with NULL for pointers.
Feb 22 2017, 2:36 AM
pfg closed D9694: sys: replace most zeros with NULL for pointers. by committing rS314068: sys: Replace zero with NULL for pointers..
Feb 22 2017, 2:36 AM
pfg added inline comments to D9694: sys: replace most zeros with NULL for pointers..
Feb 22 2017, 2:04 AM

Feb 21 2017

pfg updated the diff for D9694: sys: replace most zeros with NULL for pointers..

Remove some changes, at least one false positive.

Feb 21 2017, 3:46 PM
pfg added a reviewer for D9694: sys: replace most zeros with NULL for pointers.: cem.
Feb 21 2017, 12:33 AM

Feb 20 2017

pfg retitled D9694: sys: replace most zeros with NULL for pointers. from to sys: replace most zeros with NULL for pointers..
Feb 20 2017, 6:26 PM
pfg committed rS313982: sys/dev: Replace zero with NULL for pointers..
sys/dev: Replace zero with NULL for pointers.
Feb 20 2017, 3:43 AM
pfg committed rS313981: MFC r312997, r313002:.
MFC r312997, r313002:
Feb 20 2017, 3:34 AM
pfg committed rS313980: MFC r312992:.
MFC r312992:
Feb 20 2017, 3:26 AM

Feb 19 2017

pfg committed rS313971: MFC r313819:.
MFC r313819:
Feb 19 2017, 9:10 PM
pfg committed rS313970: MFC r313819:.
MFC r313819:
Feb 19 2017, 9:08 PM

Feb 17 2017

pfg committed rS313897: ext2fs: Remove unused assignment..
ext2fs: Remove unused assignment.
Feb 17 2017, 8:56 PM
pfg committed rS313891: sort(1): Remove unneeded initializations..
sort(1): Remove unneeded initializations.
Feb 17 2017, 7:53 PM

Feb 16 2017

pfg committed rS313819: Remove outdated claim..
Remove outdated claim.
Feb 16 2017, 8:31 PM
pfg committed rS313818: Small inclusion guard comment fix..
Small inclusion guard comment fix.
Feb 16 2017, 8:28 PM

Feb 10 2017

pfg committed rS313554: Clean redundant MIN/MAX declarations in some HighPoint drivers..
Clean redundant MIN/MAX declarations in some HighPoint drivers.
Feb 10 2017, 3:19 PM

Feb 9 2017

pfg accepted D9007: Add regression tests for indent(1).
Feb 9 2017, 11:23 PM

Feb 7 2017

pfg added a comment to D9007: Add regression tests for indent(1).
In D9007#195939, @pstef wrote:
In D9007#193594, @ngie wrote:

Thanks for the explanations -- please provide this in your commit message.

Cheers!

Sorry to bother you, but I have some doubts and I'd prefer to ask here first than to discuss on -committers when it's too late.

It's certainly better to ask early, no need to apologize.

Feb 7 2017, 9:15 PM

Feb 6 2017

pfg committed rS313321: MFC r312667:.
MFC r312667:
Feb 6 2017, 5:27 AM

Feb 3 2017

pfg committed rS313165: MFC r312001:.
MFC r312001:
Feb 3 2017, 4:36 PM
pfg committed rS313164: MFC r312001:.
MFC r312001:
Feb 3 2017, 4:35 PM
pfg committed rS313160: resolvconf: restore RESTARTCMD=, CMD1=, CMD2= and sed pattern as before..
resolvconf: restore RESTARTCMD=, CMD1=, CMD2= and sed pattern as before.
Feb 3 2017, 4:09 PM

Jan 30 2017

pfg committed rS313002: MFV 312999:.
MFV 312999:
Jan 30 2017, 11:01 PM
pfg committed rS313000: Tag import of libedit 2016-03-21..
Tag import of libedit 2016-03-21.
Jan 30 2017, 10:41 PM
pfg committed rS312999: Import libedit 2016-03-21.
Import libedit 2016-03-21
Jan 30 2017, 10:36 PM
pfg committed rS312997: MFV r312996:.
MFV r312996:
Jan 30 2017, 10:12 PM
pfg committed rS312996: Import libedit 2016-02-27.
Import libedit 2016-02-27
Jan 30 2017, 9:06 PM
pfg committed rS312992: MFV r312970:.
MFV r312970:
Jan 30 2017, 4:33 PM

Jan 29 2017

pfg committed rS312971: Tag openresolv 3.9.0..
Tag openresolv 3.9.0.
Jan 29 2017, 10:38 PM
pfg committed rS312970: Import openresolv 3.9.0.
Import openresolv 3.9.0
Jan 29 2017, 10:33 PM