[Sheepdog] [PATCH 5/5] Remove unused vosts structure

Steven Dake sdake at redhat.com
Tue May 18 06:41:45 CEST 2010


On Tue, 2010-05-18 at 10:13 +0900, FUJITA Tomonori wrote:
> On Mon, 17 May 2010 13:56:42 -0700
> Steven Dake <sdake at redhat.com> wrote:
> 
> > The vosts structure is unused and not set, resulting in a potential segfault
> > if the eprintf code is executed.  Another alternative is to set vosts[x] but
> > I am not certain what it should be set to.
> > 
> > Signed-off-by: Steven Dake <sdake at redhat.com>
> > ---
> >  sheep/sdnet.c |    5 ++---
> >  1 files changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/sheep/sdnet.c b/sheep/sdnet.c
> > index f1ce8ea..79ba3ff 100644
> > --- a/sheep/sdnet.c
> > +++ b/sheep/sdnet.c
> 
> The changes look fine but I can't find sheep/sdnet.c file. should be
> sheep/net.c ?

ya this is based off the automake patch i sent earlier (in my git repo).

Probably a better solution then renaming sdnet.c is figure how to build
lib/net.c and link it properly in automake.  I'll look into that.

Regards
-steve




More information about the sheepdog mailing list