From: David Bremner <david@tethera.net>
To: notmuch@notmuchmail.org
Subject: notmuch_database_index_file with relative paths.
Date: Fri, 17 Jul 2020 08:10:08 -0300 [thread overview]
Message-ID: <87sgdqo0rz.fsf@tethera.net> (raw)
[-- Attachment #1: Type: text/plain, Size: 485 bytes --]
This is a sanity check on a bug I stumbled upon while writing unit tests
for error handling in the library.
The attached test script (meant to be run from within the test subdir of
a built source tree) demonstrates trying (and failing) to index a
relative path.
Am I missing something here, or does _notmuch_message_file_open_ctx (or
something that calls it) need to check for absolute paths and prepend
the mail root if not?
I'm just a bit surprised that no-one hit this before.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: reproducer.sh --]
[-- Type: text/x-sh, Size: 1281 bytes --]
#!/usr/bin/env bash
test_description="notmuch_database_* API"
. $(dirname "$0")/test-lib.sh || exit 1
add_email_corpus
test_begin_subtest "building database"
test_expect_success "NOTMUCH_NEW"
cat <<EOF > c_head
#include <stdio.h>
#include <notmuch.h>
#include <notmuch-test.h>
#include <talloc.h>
int main (int argc, char** argv)
{
notmuch_database_t *db;
notmuch_status_t stat;
char *msg = NULL;
stat = notmuch_database_open_verbose (argv[1], NOTMUCH_DATABASE_MODE_READ_WRITE, &db, &msg);
if (stat != NOTMUCH_STATUS_SUCCESS) {
fprintf (stderr, "error opening database: %d %s\n", stat, msg ? msg : "");
exit (1);
}
EOF
cat <<'EOF' > c_tail
if (stat) {
const char *stat_str = notmuch_database_status_string (db);
if (stat_str)
fputs (stat_str, stderr);
}
}
EOF
generate_message '[filename]=relative_path'
test_subtest_known_broken
test_begin_subtest "index file (relative path)"
cat c_head - c_tail <<'EOF' | test_C ${MAIL_DIR}
{
notmuch_message_t *msg;
stat = notmuch_database_index_file (db, "relative_path", NULL, &msg);
printf ("%d\n", stat == NOTMUCH_STATUS_SUCCESS);
}
EOF
cat <<EOF > EXPECTED
== stdout ==
1
== stderr ==
EOF
test_expect_equal_file EXPECTED OUTPUT
test_done
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
next reply other threads:[~2020-07-17 11:10 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-17 11:10 David Bremner [this message]
2020-07-19 13:56 ` notmuch_database_index_file with relative paths David Bremner
2020-07-25 11:22 ` David Bremner
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
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87sgdqo0rz.fsf@tethera.net \
--to=david@tethera.net \
--cc=notmuch@notmuchmail.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.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
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).