Bug 437652
Summary: | pick fails with field name "From ...@bounce.linkedin.com" exceeds 127 bytes | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | John Heidemann <johnh> | ||||||
Component: | nmh | Assignee: | Josh Bressers <bressers> | ||||||
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | low | ||||||||
Version: | 8 | ||||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | i386 | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2008-03-18 12:39:26 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Description
John Heidemann
2008-03-15 19:54:21 UTC
Created attachment 298159 [details]
patch to double buffer size (NAMESZ)
Makes error go away,
but consumes more memory.
Created attachment 298160 [details]
modify the spec to apply the patch (doesn't rev version, though)
modify the spec to apply the patch (doesn't rev version, though)
Thanks for the bug report! I'm unable to reproduce this. Your error message seems to be coming from pick, and your reproducer tells me to run scan +inbox (which worked for me). Can you give me your pick flags that trigger this? Thanks. Josh wrote: >Ah ha, I now see the problem! > >It looks like you're trying to view an mbox as a mh message. the From is >missing a :, and the line doesn't look like a normal From header. > >Try to inc that file into your inbox, and you should be OK. The >question now becomes, how did it get there. Ok, two corrections. First, my original statement about how to reproduce the problem was incorrect. The exact command that reprodcues the problme is: pick -after -1 +savebox -sequence test Giving these kind of messages on a system without the patch I made available: pick: field name "From notif+qSVakQSqxx...(stuff omitted)...@bounce.linkedin.com Mon Mar " exceeds 127 bytes pick: format error in message 169 88 hits Second, I can confirm that the message IS in an mh folder. However, I agree that colon-less From is an mbox. It looks like this came from procmail, which although it supports dropping into MH folders, may not do it quite correctly. If I actually inc it, rather than let procmail deliver it, then nmh rewrites the From into a Return-path with a colon that does not trigger the error. I will try and follow up this bug with procmail. (If you want to make mh robust to this incorrectness of someone else, it looks like a 2-line change to m_getfld.c will do so. Please let me know if you're interested.) Thanks for the help. (If you agree, than this bug report should be closed NOTABUG.) Yes, I shall close this NOTABUG, and I plan to investigate this properly upstream. I plan to see about dynamically allocating the variable in question, rather than using a stack array. |