[devel] [PATCH 0 of 1] Review Request for dtm: Don't use NULL pointers or uninitialized data in error paths [#526]

2013-07-31 Thread Anders Widell
Summary: dtm: Don't use NULL pointers or uninitialized data in error paths [#526] Review request for Trac Ticket(s): 526 Peer Reviewer(s): Mahesh Pull request to: Affected branch(es): opensaf-4.2.x, opensaf-4.3.x, default(4.4) Development branch: default

Re: [devel] [PATCH 1 of 1] build: add /usr/bin/killall to sudoers list in rpm spec [#451]

2013-07-31 Thread Anders Widell
Ack (not tested). Note that this does not completely solve the problem reported in ticket 451. amfwd must check if it is running with user id zero, and if not, add the string sudo to the beginning of the command line when calling the killall program. regards, Anders Widell 2013-07-31 14:46,

[devel] [PATCH 1 of 1] osaf: support killall cmd as non-root user for amfwdog [#451]

2013-07-31 Thread mathi . naickan
00-README.conf| 3 ++- opensaf.spec.in | 2 +- osaf/services/saf/avsv/amfwdog/amf_wdog.c | 12 ++-- 3 files changed, 13 insertions(+), 4 deletions(-) amfwd uses the killall program to send the ABRT signal to amfnd. This works if

Re: [devel] [PATCH 1 of 1] dtm: Don't use NULL pointers or uninitialized data in error paths [#526]

2013-07-31 Thread A V Mahesh
Anders Widell, ACK from me and thanks for the patch . -AVM On 7/31/2013 3:57 PM, Anders Widell wrote: osaf/services/infrastructure/dtms/dtm/dtm_node_sockets.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Fixed two bugs where LOG_ER() was called with illegal parameters in