Mailing List Archive

typos and unused variables
Hi,

while packaging a current SVN snapshot for Debian, our qa tools called
my attention to a few typos and unused variables, and I'd like to share
the patches.

Florian


--- a/isakmp-pkt.c
+++ b/isakmp-pkt.c
@@ -816,7 +816,7 @@
}

DEBUG(3, printf("BEGIN_PARSE\n"));
- DEBUG(3, printf("Recieved Packet Len: %zu\n", data_len));
+ DEBUG(3, printf("Received Packet Len: %zu\n", data_len));
fetchn(r->i_cookie, ISAKMP_COOKIE_LENGTH);
hex_dump("i_cookie", r->i_cookie, ISAKMP_COOKIE_LENGTH, NULL);
fetchn(r->r_cookie, ISAKMP_COOKIE_LENGTH);
--- a/vpnc.8.template
+++ b/vpnc.8.template
@@ -176,7 +176,7 @@
make sure that files with colliding hashes have different names. The number
can be derived from the certificate file itself:
.P
-openssl x509 -subject_hash -noout -in /etc/vpnc/vpn-example-com.pem
+openssl x509 \-subject_hash \-noout \-in /etc/vpnc/vpn\-example\-com.pem

See also the
.B \-\-print\-config
@@ -241,7 +241,7 @@
License can be found in /usr/share/common\-licenses/GPL.
.SH "SEE ALSO"
.BR pcf2vpnc (1),
-.BR cisco-decrypt (1),
+.BR cisco\-decrypt (1),
.BR ip (8),
.BR ifconfig (8),
.BR route (1),
--- a/config.c
+++ b/config.c
@@ -319,7 +319,7 @@
"Script ",
"<command>",
"command is executed using system() to configure the interface,\n"
- "routing and so on. Device name, IP, etc. are passed using enviroment\n"
+ "routing and so on. Device name, IP, etc. are passed using environment\n"
"variables, see README. This script is executed right after ISAKMP is\n"
"done, but before tunneling is enabled. It is called when vpnc\n"
"terminates, too\n",
--- a/tunip.c
+++ b/tunip.c
@@ -649,12 +649,10 @@
* Process non-IP packets
* Return 1 if packet has been processed, 0 otherwise
*/
-static int process_non_ip(struct sa_block *s, uint8_t *frame)
+static int process_non_ip(uint8_t *frame)
{
struct ether_header *eth = (struct ether_header *) frame;

- s = NULL; /* unused */
-
if (ntohs(eth->ether_type) != ETHERTYPE_IP) {
/* drop non-ip traffic */
return 1;
@@ -684,7 +682,7 @@
if (process_arp(s, start)) {
return;
}
- if (process_non_ip(s, start)) {
+ if (process_non_ip(start)) {
return;
}
pack -= ETH_HLEN;
--- a/enum2debug.pl
+++ b/enum2debug.pl
@@ -6,8 +6,6 @@
use warnings;

my $in_enum = 0;
-my $element;
-my $arrayname;

print STDERR << 'EOF';
/* Automatically generated with enum2debug.pl: Don't edit! */

_______________________________________________
vpnc-devel mailing list
vpnc-devel@unix-ag.uni-kl.de
https://lists.unix-ag.uni-kl.de/mailman/listinfo/vpnc-devel
http://www.unix-ag.uni-kl.de/~massar/vpnc/
Re: typos and unused variables [ In reply to ]
On Fri, Jan 13, 2012 at 8:03 AM, Florian Schlichting
<fschlich@zedat.fu-berlin.de> wrote:
> Hi,
>
> while packaging a current SVN snapshot for Debian, our qa tools called
> my attention to a few typos and unused variables, and I'd like to share
> the patches.
>
> Florian

Pushed!

Thanks,
Antonio
_______________________________________________
vpnc-devel mailing list
vpnc-devel@unix-ag.uni-kl.de
https://lists.unix-ag.uni-kl.de/mailman/listinfo/vpnc-devel
http://www.unix-ag.uni-kl.de/~massar/vpnc/