I had another look at the issue and as it turns out, the
unicode-impl.hpp needed replacing due to a license problem when
uploading it to debian.

This is the comment I added when first uploading it to Debian:

```
 When scanning the sources, the version used with unicode-impl.hpp has
 a copyright header that is problematic with DFSG. There are more recent
 versions of this file that have an updated copyright that are compliant
 with DFSG. The upstream source has been adjusted by removing that file
 and this patch inserts an updated copy. See debian/README.source
```

and debian/README.source

```
During initial packaging, lintian picked up on a copy/not change licence
in unicode-impl.hpp.

The sources indicate that the code was fetched from  a web archive
location (ConvertUTF.h).

https://web.archive.org/web/20041122041550/http://www.unicode.org/Public/PROGRAMS/CVTUTF/

Searching further references, llvm has aa more recent copy that seems
to include an updated copyright file that allows it to be in main:

https://llvm.org/doxygen/ConvertUTF_8h_source.html

The implementation that was in the upstream release has been replaced by
the more recent copy from llvm (and adjusted to inline in a header file).

The original version has been removed in the DFSG source archive.
```

As such, the bug is on my side (the adjusted copy of
unicode-impl.hpp). I have added boundary checks.


_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to