From 0b14df43a8a6258de6884ff10b0add963b31b28d Mon Sep 17 00:00:00 2001 From: Frantisek Hrbata Date: Thu, 10 Dec 2020 17:20:31 +0100 Subject: [PATCH] mrlib: really verify the revs in verify_commits Signed-off-by: Frantisek Hrbata --- mrlib.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/mrlib.py b/mrlib.py index 80a8d74..1ade76c 100755 --- a/mrlib.py +++ b/mrlib.py @@ -157,15 +157,19 @@ def verify_comments(comments, refs): return list(revs) def verify_commits(mrcommits, commits): + revs = set() + for commit in commits: + rev = verify_rev(commit) + revs.add(rev) + mrc = set(mrcommits) mrc.add(rev_zero) - c = set(commits) - diff = c - mrc + + diff = revs - mrc if diff: sys.exit("Invalid commits: {}".format(" ".join(diff))) - return commits - + return list(revs) def add_tag(tags, mr, commit, tag, user, value): if mr not in tags: -- 1.7.10.4