18 Jun
2018
18 Jun
'18
8:29 a.m.
On Fri, Jun 15, 2018 at 03:49:51PM -0400, John Ferlan wrote:
Commit aad3a0b5f altered virObjectEventStateQueueRemote to move the "if (!event) return" call added in the previous commit 031eb8f6 to virObjectEventStateQueue. Neither commit altered the function prototype which used ATTRIBUTE_NONNULL(2).
This caused Coverity build problems. Since @event is now checked, just remove the ATTRIBUTE_NONNULL check from both prototypes.
Signed-off-by: John Ferlan <jferlan@redhat.com> --- src/conf/object_event.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Although a Coverity build breaker, that doesn't affect everyone so I'll wait for R-By or ACK for push.
Reviewed-by: Ján Tomko <jtomko@redhat.com> Jano