Re: [Dovecot] [PATCH] Bad boundary check in client_find_namespace

2012-02-09 Thread Christoph Bußenius
Hi, I'm glad to see my report finally arrive, thank you :) On 09.02.2012 04:02, Timo Sirainen wrote: Fixed now slightly differently than you: No problem – I agree that my code was a bit kludgy. I noticed that my original mail might be a bit unclear: > while trying to investigate the bug I

Re: [Dovecot] [PATCH] Bad boundary check in client_find_namespace

2012-02-08 Thread Timo Sirainen
Hi, On 26.9.2011, at 19.56, Christoph Bussenius wrote: >/* make sure two hierarchy separators aren't next to each others */ >for (p = storage_name+1; *p != '\0'; p++) { > if (p[0] == ns->real_sep && p[-1] == ns->real_sep) { > client_send_tagline(cmd, "NO Invalid mailbo

[Dovecot] [PATCH] Bad boundary check in client_find_namespace

2011-09-26 Thread Christoph Bussenius
Hi, while trying to investigate the bug I reported last week, I found that there is a broken boundary check in client_find_namespace in src/imap/imap-commands-util.c. The code is: /* make sure two hierarchy separators aren't next to each others */ for (p = storage_name+1; *p != '\0'; p++