Improved error handling for corrupt intensity profiles

loadData.c:661:15: warning: Access to field 'value' results in a dereference of a null pointer (loaded from variable 'pIRec')
                            pIRec->value = tmp;
                            ~~~~~  ^
loadData.c:682:19: warning: The right operand of '>' is a garbage value
                    if (++count > size) {
                                ^ ~~~~

Found-by: clang static analyzer
Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

1 files changed, 5 insertions, 0 deletions

diff --git a/loadData.c b/loadData.c
index 340fd96..aac86bd 100644
--- a/loadData.c
+++ b/loadData.c
@@ -634,6 +634,11 @@ ProcessIProfile(FILE *stream, XDCCC_Correction *pCorrection)
 		state = 0;
 		break;
 	      case DATA :
+		if(pIRec == NULL) {
+		    fprintf(stderr, "Line %d: Invalid Intensity Profile -- Premature %s\n",
+		            linenum, keyword);
+		    return (0);
+		}
 		do {
 		    /********************************************************
 		     * Note: tableType should only be 0 or 1 at this point.