public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* Bug fix for a compilation error
@ 2009-08-05 10:28 Maran
  2009-08-05 13:39 ` Mark Wielaard
  0 siblings, 1 reply; 2+ messages in thread
From: Maran @ 2009-08-05 10:28 UTC (permalink / raw)
  To: systemtap, Ananth N Mavinakayanahalli

Hi,
Here is the bug fix for the compilation error caused in commit 
release-0.9.9-91-g034ce93.
The error caused is,
runtime/staprun/staprun_funcs.c:509: error: ‘check_signature_rc’ 
undeclared (first use in this function)


--- a/runtime/staprun/staprun_funcs.c 2009-08-05 10:43:24.000000000 +0200
+++ b/runtime/staprun/staprun_funcs.c 2009-08-05 08:55:45.000000000 +0200
@@ -450,8 +450,8 @@ check_groups (void)
int check_permissions(void)
{
int check_groups_rc;
-#if HAVE_NSS
int check_signature_rc = 0;
+#if HAVE_NSS

/* Attempt to verify the module against its signature. Return failure
if the module has been tampered with (altered). */




Regards,
Maran

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Bug fix for a compilation error
  2009-08-05 10:28 Bug fix for a compilation error Maran
@ 2009-08-05 13:39 ` Mark Wielaard
  0 siblings, 0 replies; 2+ messages in thread
From: Mark Wielaard @ 2009-08-05 13:39 UTC (permalink / raw)
  To: Maran; +Cc: systemtap, Ananth N Mavinakayanahalli

Hi Maran,

On Wed, 2009-08-05 at 15:58 +0530, Maran wrote:
> Here is the bug fix for the compilation error caused in commit 
> release-0.9.9-91-g034ce93.
> The error caused is,
> runtime/staprun/staprun_funcs.c:509: error: ‘check_signature_rc’ 
> undeclared (first use in this function)

Ah, yes, check_signature_rc is used outside the #if HAVE_NSS block also.
Applied you fix as follows:

commit 583cad3143755445bb860f1f1e61e0f56befd5f7
Author: Maran <maranp@linux.vnet.ibm.com>
Date:   Wed Aug 5 15:36:01 2009 +0200

    Fix compile error when not HAVE_NSS with staprun.
    
    * runtime/staprun/staprun_funcs.c (check_permissions): Declare
      check_signature_rc outside HAVE_NSS block.

Thanks,

Mark

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-08-05 13:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-05 10:28 Bug fix for a compilation error Maran
2009-08-05 13:39 ` Mark Wielaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).