psDBPackRow() incorrectly handles most "META" types

[jhoblitt]

1) we are not planning to store 'globs' of binary data (at this point) in the
database. Support for everything other then PS_META_STR should be removed from
this section.

2) non-string types have strlen() called on them, see 1

3) non-string types are tested to see if they are null by comparing them to '\0'
instead of 'NULL', see 1

} else if ((item->type == PS_META_STR)

(item->type ==

PS_META_VEC)

(item->type == PS_META_IMG)

(item->type ==

PS_META_HASH)

(item->type == PS_META_LOOKUPTABLE)

(item->type ==

PS_META_JPEG)

(item->type == PS_META_PNG)

(item->type ==

PS_META_ASTROM)

(item->type == PS_META_UNKNOWN)

(item->type ==

PS_TYPE_PTR)) {

bind[i].buffer_length = (unsigned long)strlen((char *)item->data.V);

bind[i].length = &bind[i].buffer_length;

bind[i].buffer = item->data.V;
bind[i].is_null = *(char *)item->data.V == '\0'

? (my_bool *)&isNull
: NULL;

[aaron.culliney@…]

I believe this is now fixed.

[jhoblitt]

Verified by source inspection. Thanks.

[Paul Price]

Bugs have been fixed... closing.