Bug 71667 - [patch] cleanup of the usr.sbin/bootparamd code
Summary: [patch] cleanup of the usr.sbin/bootparamd code
Status: Open
Alias: None
Product: Base System
Classification: Unclassified
Component: bin (show other bugs)
Version: 5.3-BETA3
Hardware: Any Any
: Normal Affects Some People
Assignee: freebsd-bugs mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-12 18:40 UTC by Dan Lukes
Modified: 2017-11-05 21:04 UTC (History)
1 user (show)

See Also:
ngie: mfc-stable9-
ngie: mfc-stable10?
ngie: mfc-stable11?


Attachments
file.diff (870 bytes, patch)
2004-09-12 18:40 UTC, Dan Lukes
no flags Details | Diff
callbootd.c.diff (1.18 KB, patch)
2008-09-04 19:10 UTC, Rebecca Cran
no flags Details | Diff
bootparamd.c.diff (1.43 KB, patch)
2008-09-04 23:01 UTC, Rebecca Cran
no flags Details | Diff
main.c.diff (615 bytes, patch)
2008-09-04 23:01 UTC, Rebecca Cran
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Lukes 2004-09-12 18:40:26 UTC
	There are more than 5000 warnings issued during "make buildworld".
Some of them are false positives, but some of them are sign of true errors.

	Nobody is upset by warnings due it's amount, so some errors remain
uncorrected.

	I want to cleanup the code-base from warnings, so warnings will
become "attention mark" again.

usr.sbin/bootparamd/callbootd/callbootd.c:83: warning: 'clnt' might be used uninitialized in this function
bootparam_prot_clnt.c:24: warning: implicit declaration of function `memset'

How-To-Repeat: 	N/A
Comment 1 Rebecca Cran freebsd_committer 2008-09-04 19:10:13 UTC
The bootparamd code needs quite a few more changes to make it compile
with WARNS=6 with GCC 4.  I've attached a patch for
callbootd/callbootd.c, but changes are also needed for
bootparamd/bootparamd.c

-- 
Bruce Cran
Comment 2 Rebecca Cran freebsd_committer 2008-09-04 23:01:35 UTC
I have attached a patch which fixes some of the problems with
bootparamd/bootparamd.c and bootparamd/main.c.   More changes are
required to bootparamd.c for it to compile with WARNS=6
though.

-- 
Bruce Cran
Comment 3 commit-hook freebsd_committer 2015-11-10 11:20:20 UTC
A commit references this bug:

Author: ngie
Date: Tue Nov 10 11:19:37 UTC 2015
New revision: 290645
URL: https://svnweb.freebsd.org/changeset/base/290645

Log:
  Fix some trivial warnings with bootparamd/main.c

  - Convert K&R to something a bit less ancient
  - Remove an incorrect, duplicate prototype for bootparamprog_1(..)

  MFC after: 1 week
  PR: 71667
  Submitted by: bcran
  Sponsored by: EMC / Isilon Storage Division

Changes:
  head/usr.sbin/bootparamd/bootparamd/main.c
Comment 4 commit-hook freebsd_committer 2015-11-19 00:15:18 UTC
A commit references this bug:

Author: ngie
Date: Thu Nov 19 00:14:23 UTC 2015
New revision: 291042
URL: https://svnweb.freebsd.org/changeset/base/291042

Log:
  MFC r290645:

  Fix some trivial warnings with bootparamd/main.c

  - Convert K&R to something a bit less ancient
  - Remove an incorrect, duplicate prototype for bootparamprog_1(..)

  PR: 71667
  Submitted by: bcran
  Sponsored by: EMC / Isilon Storage Division

Changes:
_U  stable/10/
  stable/10/usr.sbin/bootparamd/bootparamd/main.c
Comment 5 commit-hook freebsd_committer 2015-11-19 00:19:22 UTC
A commit references this bug:

Author: ngie
Date: Thu Nov 19 00:19:09 UTC 2015
New revision: 291043
URL: https://svnweb.freebsd.org/changeset/base/291043

Log:
  MFstable/10 r291042:

  MFC r290645:

  Fix some trivial warnings with bootparamd/main.c

  - Convert K&R to something a bit less ancient
  - Remove an incorrect, duplicate prototype for bootparamprog_1(..)

  PR: 71667
  Submitted by: bcran
  Sponsored by: EMC / Isilon Storage Division

Changes:
_U  stable/9/
_U  stable/9/usr.sbin/
_U  stable/9/usr.sbin/bootparamd/
  stable/9/usr.sbin/bootparamd/bootparamd/main.c
Comment 6 commit-hook freebsd_committer 2017-08-17 20:38:07 UTC
A commit references this bug:

Author: ngie
Date: Thu Aug 17 20:37:02 UTC 2017
New revision: 322635
URL: https://svnweb.freebsd.org/changeset/base/322635

Log:
  Fix WARNS

  - Remove ad hoc inet_ntoa prototype declaration; it's already handled
    by the included headers.
  - De-K&Rify the function prototypes for eachres_whoami(..),
    eachres_getfile(..), and main(..).

  MFC after:	1 week
  PR:		71667
  Tested with:	clang (5.0), gcc (4.2.1, 5)

Changes:
  head/usr.sbin/bootparamd/callbootd/callbootd.c
Comment 7 Enji Cooper freebsd_committer 2017-08-17 20:42:11 UTC
A lot of these warnings were fixed in the clang integration project BTW.
Comment 8 Dan Lukes 2017-08-18 07:20:40 UTC
(In reply to Ngie Cooper from comment #7)

May be, but despite of it, there are 10648 warnings fired during compilation of 11.0-RELEASE-p12.

But I'm not going to analyze them in attempt to identify possible race conditions and bug. 

I spent so much time to something like it 13 years ago and I'm not willing to do it again as it seems to be pure waste of time.

Even those PR related to true bugs and race conditions (e.g. no just meaningless warning) has been left untouched for years. 

It seems commiters doesn't support/welcome neither bug hunting nor cleanup of code in attempt to help bug hunters.

But thank you very much for the time you spent to process this 13 years old PR.
Comment 9 commit-hook freebsd_committer 2017-09-30 20:51:17 UTC
A commit references this bug:

Author: ngie
Date: Sat Sep 30 20:50:32 UTC 2017
New revision: 324142
URL: https://svnweb.freebsd.org/changeset/base/324142

Log:
  MFC r322635:

  Fix WARNS

  - Remove ad hoc inet_ntoa prototype declaration; it's already handled
    by the included headers.
  - De-K&Rify the function prototypes for eachres_whoami(..),
    eachres_getfile(..), and main(..).

  PR:		71667
  Tested with:	clang (5.0), gcc (4.2.1, 5)

Changes:
_U  stable/10/
  stable/10/usr.sbin/bootparamd/callbootd/callbootd.c
Comment 10 Enji Cooper freebsd_committer 2017-11-05 21:04:04 UTC
I have no way to test this. Untaking.