cygcheck patch (updated)

[Matt <matt at use dot net>]

I forgot to mention that I tested this on both Win98 and NT4, gcov
reported 80% code coverage with my test cases. I ran the test cases on the
old version and the new version, comparing the output (which was the same,
save the timestamp printed and the path to the executable). The timings
(using time) is about the same.

Test cases were:
1. cygcheck (no args)
2. cygcheck -h -v -s -r
3. cygcheck -v -h -k (press lotsa keys, then q)


Changelog:

2000-12-07  Matt Hargett  <matt@use.net>

	* utils/cygcheck.cc (keyeprint): Moved declaration before other
	functions so it could be used by all functions to report error
	messages. Added comment.

	(add_path): Initial uses of pointers are now checked for
	NULL. This includes pointers from malloc() and pointers 
 	passed as parameters.

	(find_on_path): Ditto.

	(rva_to_offset): Ditto.

	(init_paths): Added checking for return values of Win32 calls.

	(get_dword): Ditto.

	(get_word): Ditto.

	(dll_info): Ditto, also NULL pointer checks.

	(scan_registry): Ditto.

	(check_keys): Ditto.

        (dump_sysinfo): Ditto. Added default case to switch. Added error
	reporting if GetVolumeInformation fails, except when it
	returns ERROR_NOT_READY. ERROR_NOT_READY is returned when
	removeable drives are empty.

	(track_down): Added checking for NULL pointers and 
	return values of Win32 calls. Corrected small memory leak.

cygcheck.cc-patch.gz