I mean, the comment "this should not happen" implies that this code path should never be hit, but this patch is now using it for legitimate error-reporting purposes.

The fact that the comment exists implies that it's expected that this sort of error is handled/reported elsewhere. Of course, it may be that this expectation is now incorrect, and that this is the correct codepath for the error. (In which case, the comment should be removed.)