Launchpad has imported 9 comments from the remote bug at
http://bugs.exim.org/show_bug.cgi?id=2821.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2021-10-06T14:22:37+00:00 Fiedler-x wrote:

The original author of this comment is ltning-e...@anduin.net.


If exiqgrep is invoked with -r or -f but without specifying a regular
expression -  e.g. like
  exiqgrep -i -r | xargs exim -Mrm
all enqueued are deleted.

Help output states
  Selection criteria:
        -f <regexp>     Match sender address sender (field is "< >" wrapped)
  ...
which indicates that <regexp> is a required criteria.

I presume the absence of <regexp> is treated as an empty regex, matching
everything - but I would consider this a bug, or at least unexpected behaviour.

Needless to say the fallout of discovering this was somewhat significant. User
error, yes, but not immediately obvious.
    

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/0

------------------------------------------------------------------------
On 2021-10-20T14:53:19+00:00 Fiedler-x wrote:


--- Bug imported by fied...@schlittermann.de 2021-10-20 14:53 UTC  ---

This bug was previously known as _bug_ 2821 at
https://bugs.exim.org/show_bug.cgi?id=2821

Unknown bug field "comment_sort_order" encountered while moving bug
   <comment_sort_order>oldest_to_newest</comment_sort_order>
Unknown platform unknown. Setting to default platform "".
The original reporter of this bug does not have
   an account here. Reassigning to the person who moved
   it here: fied...@schlittermann.de.
   Previous reporter was ltning-e...@anduin.net.


Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/1

------------------------------------------------------------------------
On 2021-10-21T13:19:44+00:00 Git-p wrote:

Git commit:
https://git.exim.org/exim.git/commitdiff/df618101a5ea15dc90c4a2968798ef2be9dba16f

commit df618101a5ea15dc90c4a2968798ef2be9dba16f
Author:     Jeremy Harris <jgh146...@wizmail.org>
AuthorDate: Mon Oct 18 11:01:47 2021 +0100
Commit:     Jeremy Harris <jgh146...@wizmail.org>
CommitDate: Mon Oct 18 11:01:47 2021 +0100

    Exiqgrep: check arg parsing.  Bug 2821
---
 src/src/exiqgrep.src | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/src/exiqgrep.src b/src/src/exiqgrep.src
index c8762df..04602da 100644
--- a/src/src/exiqgrep.src
+++ b/src/src/exiqgrep.src
@@ -53,7 +53,7 @@ if ($ARGV[0] eq '--version') {
         exit 0;
 }
 
-getopts('hf:r:y:o:s:C:zxlibRcaG:',\%opt);
+if (!getopts('hf:r:y:o:s:C:zxlibRcaG:',\%opt) { &help; exit;}
 if ($ARGV[0]) { &help; exit;}
 if ($opt{h}) { &help; exit;}
 if ($opt{a}) { $eargs = '-bp'; }

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/2

------------------------------------------------------------------------
On 2021-10-21T13:27:23+00:00 Git-p wrote:

Git commit:
https://git.exim.org/exim.git/commitdiff/df618101a5ea15dc90c4a2968798ef2be9dba16f

commit df618101a5ea15dc90c4a2968798ef2be9dba16f
Author:     Jeremy Harris <jgh146...@wizmail.org>
AuthorDate: Mon Oct 18 11:01:47 2021 +0100
Commit:     Jeremy Harris <jgh146...@wizmail.org>
CommitDate: Mon Oct 18 11:01:47 2021 +0100

    Exiqgrep: check arg parsing.  Bug 2821
---
 src/src/exiqgrep.src | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/src/exiqgrep.src b/src/src/exiqgrep.src
index c8762df..04602da 100644
--- a/src/src/exiqgrep.src
+++ b/src/src/exiqgrep.src
@@ -53,7 +53,7 @@ if ($ARGV[0] eq '--version') {
         exit 0;
 }
 
-getopts('hf:r:y:o:s:C:zxlibRcaG:',\%opt);
+if (!getopts('hf:r:y:o:s:C:zxlibRcaG:',\%opt) { &help; exit;}
 if ($ARGV[0]) { &help; exit;}
 if ($opt{h}) { &help; exit;}
 if ($opt{a}) { $eargs = '-bp'; }

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/3

------------------------------------------------------------------------
On 2021-11-20T16:51:04+00:00 Jgh146exb wrote:

Nobody commented

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/4

------------------------------------------------------------------------
On 2022-03-02T18:14:20+00:00 Eximusers-i wrote:

(In reply to Jeremy Harris from comment #4)
> Nobody commented

The applied patch introduces unbalanced parentheses.
syntax error at /usr/sbin/exiqgrep line 56, near ") {"
syntax error at /usr/sbin/exiqgrep line 56, near ";}"
syntax error at /usr/sbin/exiqgrep line 57, near ";}"
syntax error at /usr/sbin/exiqgrep line 58, near ";}"
Execution of /usr/sbin/exiqgrep aborted due to compilation errors.

cu Andreas

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/5

------------------------------------------------------------------------
On 2022-03-06T16:48:48+00:00 Jgh146exb wrote:

Oops.  We had no regression-test case.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/6

------------------------------------------------------------------------
On 2022-03-06T17:27:06+00:00 Git-p wrote:

Git commit:
https://git.exim.org/exim.git/commitdiff/42ed39da5af85552a35626348bb77e5576e18aa4

commit 42ed39da5af85552a35626348bb77e5576e18aa4
Author:     Jeremy Harris <jgh146...@wizmail.org>
AuthorDate: Sun Mar 6 14:25:13 2022 +0000
Commit:     Jeremy Harris <jgh146...@wizmail.org>
CommitDate: Sun Mar 6 16:45:03 2022 +0000

    utilities: fix exiqgrep perl syntax, add testcases.  bug 2821
    
    broken-by: df618101a5
----
 doc/doc-docbook/spec.xfpt    | 18 ++++++++++---
 src/src/exiqgrep.src         |  9 ++++---
 test/README                  |  6 +++++
 test/runtest                 | 15 +++++++++--
 test/scripts/0000-Basic/0611 |  6 +++++
 test/stdout/0611             | 62 ++++++++++++++++++++++++++++++++++++++++++++
 6 files changed, 108 insertions(+), 8 deletions(-)

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/7

------------------------------------------------------------------------
On 2022-03-20T16:25:00+00:00 Jgh146exb wrote:

No further comments seen.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/exim4/+bug/1966923/comments/8


** Changed in: exim
       Status: Unknown => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1966923

Title:
  exiqgrep -r and -f options match all messages if no

To manage notifications about this bug go to:
https://bugs.launchpad.net/exim/+bug/1966923/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to