[devel] [PATCH 0 of 1] Review Request for logsv: Fix checkpoint version handling [#688]

2014-01-27 Thread Lennart Lund
Summary: logsv: Fix checkpoint version handling [#688] Review request for Trac Ticket(s): #688 Peer Reviewer(s): mathi.naic...@oracle.com Pull request to: Affected branch(es): 4.4 Development branch: IF ANY GIVE THE REPO URL Impacted area Impact y/n

[devel] [PATCH 0 of 1] Review Request for logsv: Fix updating of log root directory when check-pointing lgs config [#711]

2014-01-27 Thread Lennart Lund
Summary: logsv: Fix updating of log root directory when check-pointing lgs config [#711] Review request for Trac Ticket(s): #711 Peer Reviewer(s): mathi.naic...@oracle.com Pull request to: Affected branch(es): 4.4 Development branch: IF ANY GIVE THE REPO URL

[devel] [PATCH 1 of 1] logsv: Fix updating of log root directory when check-pointing lgs config [#711]

2014-01-27 Thread Lennart Lund
osaf/services/saf/logsv/lgs/lgs_imm.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) Fix updating of lgs_cb when logRootDirectory is check pointed. diff --git a/osaf/services/saf/logsv/lgs/lgs_imm.c b/osaf/services/saf/logsv/lgs/lgs_imm.c ---

Re: [devel] [PATCH 1 of 1] amfd: fix out-of-bounds read from nodelist array [#719]

2014-01-27 Thread Nagendra Kumar
Ack Thanks -Nagu -Original Message- From: Gary Lee [mailto:gary@dektech.com.au] Sent: 24 January 2014 11:56 To: hans.nordeb...@ericsson.com; gary@dektech.com.au Cc: opensaf-devel@lists.sourceforge.net Subject: [devel] [PATCH 1 of 1] amfd: fix out-of-bounds read from

[devel] [PATCH 0 of 1] Review Request for amfnd: errors reported by valgrind (V2) [#716]

2014-01-27 Thread Gary Lee
Summary: amfnd: errors reported by valgrind (V2) [#716] Review request for Trac Ticket(s): 716 Peer Reviewer(s): Hans N, Nagendra Pull request to: Affected branch(es): default, opensaf-4.4.x Development branch: default Impacted area Impact y/n

[devel] [PATCH 1 of 1] amfnd: fix errors reported by valgrind [#716]

2014-01-27 Thread Gary Lee
osaf/services/saf/amf/amfnd/amfnd.cc|4 +- osaf/services/saf/amf/amfnd/cbq.cc |4 +- osaf/services/saf/amf/amfnd/ckpt_updt.cc| 18 +- osaf/services/saf/amf/amfnd/comp.cc | 11 +- osaf/services/saf/amf/amfnd/compdb.cc |6 +-

Re: [devel] [PATCH 1 of 2] amfd: require saAmfCtSwBundle when relpath in CLC-CLI cmds [#662]

2014-01-27 Thread Nagendra Kumar
Nack. In general, not only instantiate, but other clc commands has to be validated at amfd. Logic should be if (saAmfCtSwBundle is configured), then clc commands should not start with /. Else if (saAmfCtSwBundle is not configured), then clc commands should start with /. Thanks -Nagu