Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 17066 - pump and resolv.com
pump and resolv.com
Product: Red Hat Linux
Classification: Retired
Component: pump (Show other bugs)
i386 Linux
high Severity high
: ---
: ---
Assigned To: Erik Troan
Depends On:
  Show dependency treegraph
Reported: 2000-08-29 06:50 EDT by Michael Stefaniuc
Modified: 2008-05-01 11:37 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2000-08-30 12:50:28 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Michael Stefaniuc 2000-08-29 06:50:18 EDT
pump leaves the "search" entry in /etc/resolv.conf empty (tested on 7.0
gold,  pump-0.8.3-2).
On a 6.1 and a 6.2 the search entry was set to the "option domain-name"
from the dhcpd.
Comment 1 Michael Stefaniuc 2000-08-30 12:19:43 EDT
If the domain-name contains only one "." and the toplevel domain isn't one of
com, edu, gov, mil, org, net, int, then the "search" entry in resolv.conf is
Hier is the patch:
--- ./pump.c.search     Wed Aug 30 17:59:02 2000
+++ ./pump.c    Wed Aug 30 18:03:02 2000
@@ -164,8 +164,8 @@
     } else if (domain && !strchr(domain, '.')) {
        fprintf(f, "search %s\n", domain);
     } else if (domain) {
-       fprintf(f, "search");
-       chptr = domain;
+       fprintf(f, "search %s", domain);
+       chptr = strchr(domain, '.');
        do {
            /* If there is a single . in the search path, write it out
             * only if the toplevel domain is com, edu, gov, mil, org,
Comment 2 Bernhard Rosenkraenzer 2000-08-30 12:37:44 EDT
I've fixed this (but a bit differently) in CVS [check the comment in the code].

Leaving the bug open because whatever this behavior was supposed to accomplish
(if anything) is gone now.
Comment 3 Bernhard Rosenkraenzer 2000-08-30 12:50:26 EDT
0.8.4 (with my change) built in dist-7.1
Comment 4 Erik Troan 2000-08-30 13:45:06 EDT
A similiar change is already in recent versions of pump (including that in

Note You need to log in before you can comment on or make changes to this bug.