All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mads Martin Joergensen <mmj@mmj.dk>
To: mlmmj@mlmmj.org
Subject: Re: Bug in subscriber searching function is_subbed_in()
Date: Mon, 07 Aug 2006 09:44:57 +0000	[thread overview]
Message-ID: <20060807094457.GA54628@mmj.dk> (raw)
In-Reply-To: <44D6B83A.6030701@kite.se>

* Magnus Naeslund(k) <mag@kite.se> [Aug 07. 2006 05:49]:
> I found out that the subscribercode searches the subscriber directory 
> and tries to mmap the lockfile it created earlier which size is zero,
> hence the logentry:
> 
> /usr/bin/mlmmj-sub[25209]: subscriberfuncs.c:61: Could not mmap fd: Invalid argument
> 
> I modified the code to skip ALL files that starts with a dot, that
> takes care if . and .. directories aswell as the .$file.lock files.
> 
> Is this patch acceptable?

Not really. It means it wont search .somesubshereinthis file. But to
that bug, there's already a fix in the tree:

        /* No need to check in 0-size file */
        if(st.st_size = 0)
                return (off_t)-1;

So after ignoring . and .. we only work on files bigger than 0.

-- 
Mads Martin Joergensen, http://mmj.dk
"Why make things difficult, when it is possible to make them cryptic
 and totally illogical, with just a little bit more effort?"
                                 -- A. P. J.

      reply	other threads:[~2006-08-07  9:44 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-07  3:49 Bug in subscriber searching function is_subbed_in() Magnus Naeslund(k)
2006-08-07  9:44 ` Mads Martin Joergensen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060807094457.GA54628@mmj.dk \
    --to=mmj@mmj.dk \
    --cc=mlmmj@mlmmj.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.