Skip to Content.
Sympa Menu

freetds - [freetds] [PATCH] useless CHECK_NULP in dblib.c:tdsdbopen()

freetds AT lists.ibiblio.org

Subject: FreeTDS Development Group

List archive

Chronological Thread  
  • From: "Craig A. Berry" <craigberry AT mac.com>
  • To: FreeTDS Development Group <freetds AT lists.ibiblio.org>
  • Subject: [freetds] [PATCH] useless CHECK_NULP in dblib.c:tdsdbopen()
  • Date: Fri, 20 Mar 2009 12:52:35 -0500

The CHECK_NULP statement removed by the attached patch is not actually checking anything except whether random data from the stack is non- zero. Usually it will be, but why depend on it and what good would it do to know that?

My compiler said something about "dbproc fetched but not initialized" and I'm sure gcc and other compilers have ways to generate similar warnings.
________________________________________
Craig A. Berry
mailto:craigberry AT mac.com

"... getting out of a sonnet is much more
difficult than getting in."
Brad Leithauser

Attachment: nulp.patch
Description: Binary data






Archive powered by MHonArc 2.6.24.

Top of Page