Merged manually to tweak the commit messages a little bit, primarily to
capitalize sentences.
Thanks a lot for the patches! While we do check for memory leaks every now and
then, clearly nobody has done so for src.rpm installations in a long, long time
if ever.
--
Reply to this email directly
Closed #2289.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2289#event-7890662487
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mailing list
Rpm-
Closed #2286 as completed via #2287.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2286#event-7890654619
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rp
Merged #2287 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2287#event-7890654539
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint maili
@JetXujing commented on this pull request.
> @@ -241,6 +241,8 @@ Header headerFree(Header h)
}
h->index = _free(h->index);
}
+if (h->blob)
Thank you for your comments. I've made changes.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-soft
> You can't reference some local files in the commit message ([fix file leak
> when install src rpm which is
> URL](https://github.com/rpm-software-management/rpm/pull/2289/commits/3938cefacedbfd1ae40ff356c9d8d43e173da902)).
> You need to include the content if it is appropriate. Or describe the
@JetXujing pushed 2 commits.
310bad24176e0213af7e5333de491abb1c3a5e81 fix file leak when src rpm in URL
format is used for installation
b70d28c65e88afe96f4ce6085aa476e4d173f826 fix h->blob leak when install src rpm
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2289
@pmatilai commented on this pull request.
> @@ -241,6 +241,8 @@ Header headerFree(Header h)
}
h->index = _free(h->index);
}
+if (h->blob)
Don't bother checking for non-NULL, free() does it anyway.
--
Reply to this email directly or view it on GitHub:
https://github.co
You can't reference some local files in the commit message ([fix file leak when
install src rpm which is
URL](https://github.com/rpm-software-management/rpm/pull/2289/commits/3938cefacedbfd1ae40ff356c9d8d43e173da902)).
You need to include the content if it is appropriate. Or describe the issue.
@JetXujing pushed 2 commits.
3938cefacedbfd1ae40ff356c9d8d43e173da902 fix file leak when install src rpm
which is URL
374adb562e29c1ad736e39e07dd859decb0cd5e9 fix h->blob leak when install src rpm
--
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2289/files/1e973b50274
Closed #2276 as completed via #2290.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2276#event-7884763690
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rp
Merged #2290 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2290#event-7884763514
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint maili
@nwalfield approved this pull request.
Looks good!
```
[neal@fedora-36 _build]$ git describe
rpm-4.17.0-alpha-688-g9f3d9aa8c
[neal@fedora-36 _build]$ make check
...
263: rpmkeys -K verifylevel ok
266: rpmkeys -Kv 1ok
265: rpmkeys -K verifylevel
@nwalfield commented on this pull request.
> @@ -740,9 +770,44 @@ runroot rpmkeys -Kv /tmp/${pkg}
],
[])
AT_CLEANUP
+
+# --
+# Test pre-built corrupted package verification (corrupted header)
Great addition!
--
Reply to this email directly or view it on GitHub:
Wider exposure of the Sequoia backend taught us that OpenPGP v3 signatures are
surprisingly common still in the rpm-ecosystem, and in fact more common than v4
signatures. It's a bit surprising, considering that gnupg has defaulted to
creating v4 signatures since 1.4.8 from late 2007 gnupg2 refus
Closed #2235.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2235#event-7884363590
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mailing list
Rpm-
Reopened #2235.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2235#event-7884361166
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint mailing list
Rp
17 matches
Mail list logo