You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/1535
-- Commit Summary --
* Fixed issues find by coverity
-- File Changes --
M plugins/fapolicyd.c (4)
-- Patch Links --
> Excellent, thanks.
>
> It's annoying how the GitHub interface tends to hide the commit message,
> which is often just about as important as the code...
Totally agree...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
> Okay, very nearly good to go, I was just about to hit merge before realizing
> that the commit message is not at all the same as the PR description. Please
> explain the use-case in the commit message: basically the PR description plus
> a brief explanation of the issues involved and why this
@radosroka pushed 1 commit.
e34dbab3dcff2035ca8c0ba7a80e3454b1d429dc Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka pushed 1 commit.
9b36cd27e0442c3e8dd2eb391bed7aab237b7e90 Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka pushed 1 commit.
83b75dbe12a8d778089c01e0fd9f3e08fc0fe02e Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka pushed 1 commit.
46bae1b13ba12e9d40f70f378c6846a881047c53 Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka pushed 1 commit.
abb063fee6ac36439ce1a1a0550261b2c5d7972b Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka commented on this pull request.
> +
+static rpmRC fapolicyd_tsm_post(rpmPlugin plugin, rpmts ts, int res)
+{
+if (fapolicyd_state.ready)
+write_fifo(_state, "1", 2);
+
+return RPMRC_OK;
+}
+
+static rpmRC fapolicyd_scriptlet_pre(rpmPlugin plugin, const char *s_name,
+
@radosroka pushed 1 commit.
f64c8f5afc1f17e2e3f757a5cf12e700bebbad1d Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka pushed 1 commit.
9944f7d475c4a45535feda7e56725fb1a2ac3b02 Added fapolicyd plugin
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
@radosroka commented on this pull request.
> +
+static rpmRC fapolicyd_tsm_post(rpmPlugin plugin, rpmts ts, int res)
+{
+if (fapolicyd_state.ready)
+write_fifo(_state, "1", 2);
+
+return RPMRC_OK;
+}
+
+static rpmRC fapolicyd_scriptlet_pre(rpmPlugin plugin, const char *s_name,
+
@radosroka commented on this pull request.
> +fd = open(state->fifo_path, O_WRONLY);
+if(fd == -1) {
+rpmlog(RPMLOG_DEBUG, "Open: %s -> %s\n", state->fifo_path,
strerror(errno));
+return RPMRC_OK;
+}
+
+if (stat(state->fifo_path, ) == -1) {
+
The plugin notifies fapolicyd daemon about ongoing rpm transaction via linux
pipe.
It sends files info in %s %lu %64s\n format.
The daemon adds these records dynamically to the DB and they are considered as
trusted.
Related PRs:
14 matches
Mail list logo