<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/1503/">https://reviewboard.asterisk.org/r/1503/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks for taking care of this.</pre>
<br />
<p>- Kevin</p>
<br />
<p>On October 10th, 2011, 12:25 p.m., Tzafrir Cohen wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Tzafrir Cohen.</div>
<p style="color: grey;"><i>Updated Oct. 10, 2011, 12:25 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Recently Sam Geeraerts filed a bug with the Debian package of Asterisk regarding the license of main/sha1.c . It seems that the license of that file, as is the license of many older RFCs, does not allow modifications to the source file (except for some very specific purposes). This is a violation of the Debian Free Software Guidelines and cannot be allowed in Debian.
Luckily, RFC 3174 (from which the existing code was taken) has been replaced by the recent RFC 6234. The latter comes with a simpler license (a BSD one) and a somewhat better code. This patch updates main/sha1.c and includes/asterisk/sha1.h usinf files from that RFC:
* sha.h and sha-private.h were merged into sha1.h
* Coding guildelines fixes (in sha1.c, mostly)
* Removed an unused include of sha1.h from main/channels.c .
The code is not very different. Some functions were statified. Some magic numbers were made constants. A few more unrequired headers (for sha256.c and co.), but almost no real code changes.
sha1.c has remained basically the same for quite some time, and thus this patch can be applied as-is on top of older branches, should it be useful to anybody. Alternatively: just copy the resulting source file after the commit.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Basic testing of function SHA1.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/include/asterisk/sha1.h <span style="color: grey">(340161)</span></li>
<li>/trunk/main/channel.c <span style="color: grey">(340161)</span></li>
<li>/trunk/main/sha1.c <span style="color: grey">(340161)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/1503/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>