Added missing include files

Add some missing include files

remove postgres.h from fmgr.h creation

won't be...

Added needed include file.

Added needed include file.

Pointed out by: wieck@sapserv.debis.de

I found another bug in btree index.  Looking at the code it seems that NULL
keys are never used to build or scan a btree index (see the explain commands
in the example).  However this is not the case when a null key is retrieved
in an outer loop of a join select and used in an index scan of an inner loop.
This bug causes at least three kinds of problems:

1)  the backend crashes when it tries to compare a text string with a null.

2)  it is not possible to find tuples with null keys in a join.

3)  null is considered equal to 0 when the datum is passed by value, see
    the last query.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

has snprintf().

For sparc_solaris, add inet_aton prototype, since it isn't in the header
files included for the standard C library functions.

the standard C library.

Still submitted by:  Massimo Dal Zotto <dz@cs.unitn.it>

Submitted by: wieck@sapserv.debis.de (Jan Wieck)

              relations and transaction is aborted

Submitted by: wieck@sapserv.debis.de (Jan Wieck)

It's bug in nodeAgg.c on lines 241, 242:

                null_array = malloc(nagg);
                for (i=0;i<nagg;i++)
                    null_array[i] = 'n';
                oneTuple = heap_formtuple(tupType, tupValue, null_array);

- your query has not only aggregates but also 'group by-ed' fields and so
null_array should contain tupType->natts elements (tupType->natts > nagg in
your case).

Patch follows and it's very simple.

VAdim

My guess is that the thing had bugs, and the pfree was commented out.
The thing is probabally free'ed anyway at the end, so it was not a bad
thing.

If it does cause a bug, it will generate an error when hit, so I say
unless someone else knows, let's remove it and run the regression test.

-Bruce

Suggested by: Randy Terbush <randy@zyzzyva.com>