Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-developers-owner AT sourceware DOT cygnus DOT com Delivered-To: mailing list cygwin-developers AT sourceware DOT cygnus DOT com To: cygwin-developers AT sourceware DOT cygnus DOT com Subject: Re: cygwin_bind bug. References: <19991124192413 DOT A12159 AT cygnus DOT com> <19991124205652 DOT B15777 AT cygnus DOT com> Mime-Version: 1.0 (generated by tm-edit 7.106) Content-Type: text/plain; charset=US-ASCII From: Kazuhiro Fujieda Date: 25 Nov 1999 11:28:38 +0900 In-Reply-To: Chris Faylor's message of Wed, 24 Nov 1999 20:56:52 -0500 Message-ID: Lines: 24 X-Mailer: Gnus v5.3/Emacs 19.34 >>> On Wed, 24 Nov 1999 20:56:52 -0500 >>> Chris Faylor said: > In that case, why even use an intermediate variable? Why not just check > the results from bind and getsockname directly? There is no reason... I carelessly failed to notice there is no need of it. > Index: net.cc ... > else > { > _close (fd); > chmod (un_addr->sun_path, > (S_IFSOCK | S_IRWXU | S_IRWXG | S_IRWXO) & ~myself->umask); > - res = 0; > } It is overkill. No one can set `res' to 0. ____ | AIST Kazuhiro Fujieda | HOKURIKU School of Information Science o_/ 1990 Japan Advanced Institute of Science and Technology