diff options
| -rw-r--r-- | src/or/consdiff.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/or/consdiff.c b/src/or/consdiff.c index 65e15f79ee..a340cfd8f8 100644 --- a/src/or/consdiff.c +++ b/src/or/consdiff.c @@ -670,15 +670,15 @@ apply_ed_diff(smartlist_t *cons1, smartlist_t *diff) if (*endptr1 == ',') { end = (int)strtol(endptr1+1, &endptr2, 10); if (endptr2 == endptr1+1) { - goto error_cleanup; log_warn(LD_CONSDIFF, "Could not apply consensus diff because " "an ed command was missing a range end line number."); + goto error_cleanup; } /* Incoherent range. */ if (end <= start) { - goto error_cleanup; log_warn(LD_CONSDIFF, "Could not apply consensus diff because " "an invalid range was found in an ed command."); + goto error_cleanup; } /* We'll take <n1> as <n1>,<n1> for simplicity. */ @@ -693,6 +693,12 @@ apply_ed_diff(smartlist_t *cons1, smartlist_t *diff) goto error_cleanup; } + if (*endptr2 == '\0') { + log_warn(LD_CONSDIFF, "Could not apply consensus diff because " + "a line with no ed command was found"); + goto error_cleanup; + } + if (*(endptr2+1) != '\0') { log_warn(LD_CONSDIFF, "Could not apply consensus diff because " "an ed command longer than one char was found."); |