Yes please do, I think that's a good idea. Thanks Lennart
> -----Original Message----- > From: Mathivanan Naickan Palanivelu [mailto:[email protected]] > Sent: den 28 augusti 2014 14:14 > To: Lennart Lund > Cc: [email protected] > Subject: Re: [PATCH 1 of 1] log: Fix incorrect detection of log conf obj > invalid > [#1027] > > Ack, > > Should we change the document to say that we support only a fixed object > name "logConfig=1,safApp=safLogService"? > Today, we seem to suggest that any object of the logconfig class will work!? > > Mathi. > > ----- [email protected] wrote: > > > osaf/services/saf/logsv/lgs/lgs_imm.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > diff --git a/osaf/services/saf/logsv/lgs/lgs_imm.c > > b/osaf/services/saf/logsv/lgs/lgs_imm.c > > --- a/osaf/services/saf/logsv/lgs/lgs_imm.c > > +++ b/osaf/services/saf/logsv/lgs/lgs_imm.c > > @@ -2852,7 +2852,7 @@ const void *lgs_imm_logconf_get(lgs_logc > > lgs_conf_p = (lgs_conf_t *) lgs_conf; > > /* Check if parameters has to be fetched from IMM */ > > if (lgs_conf->logInitiated != true) { > > - if (read_logsv_config_obj()) { > > + if (read_logsv_config_obj() != SA_AIS_OK) { > > LOG_NO("No or invalid log service configuration > object"); > > read_logsv_config_environ_var(); > > } else { ------------------------------------------------------------------------------ Slashdot TV. Video for Nerds. Stuff that matters. http://tv.slashdot.org/ _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
