[PATCH] Logging of files with broken encoding

Jeremy Allison jra at samba.org
Tue Jun 9 10:10:29 MDT 2015


On Tue, Jun 09, 2015 at 01:41:42PM +0200, Ralph Böhme wrote:
> Hi,
> 
> attached is a patchset that moves a level 0 catch-all debug statement
> from behind the switch clause inside the switch clause as default
> case. This fixes an issue that resulted in the log being flooded with
> those messages in case someone puts a file with an illegal UTF8
> encoding (eg '\xA0test') in a toplevel directory:
> 
> [2015/05/12 18:17:26.865144,  0] ../lib/util/charset/convert_string.c:438(convert_string_talloc_handle)
>   Conversion error: Incomplete multibyte sequence(<A0>test)
> 
> There's already an explicit EILSEQ case with a log message at loglevel
> 3, so afaict the level 0 log statement after the switch clause doesn't
> make sense.
> 
> Otoh, I'd think it would be worthwhile logging a useful diagnostic
> with the full file patch which is what patch 2/2 does.
> 
> As a result, with both patches in place we'd write this to the logs:
> 
> [2015/06/09 13:18:51.429654,  1, pid=3461] ../source3/smbd/trans2.c:2379(smbd_dirptr_lanman2_entry)
>   Conversion error: illegal character: dir/test?
> 
> Thoughts?

Really nice cleanup - pushed !

Do you want to log a bug to get this into 4.2.next, 4.1.next ?


More information about the samba-technical mailing list