linux-kernel - Re: [PATCH] crypto: fix error flow of crypto/testmgr.c:test

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20081125.170851.97706149.davem@davemloft.net>
Date:	Tue, 25 Nov 2008 17:08:51 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	mingo@...e.hu
Cc:	herbert@...dor.apana.org.au, linux-kernel@...r.kernel.org,
	linux-crypto@...r.kernel.org
Subject: Re: [PATCH] crypto: fix error flow of crypto/testmgr.c:test_comp()

From: Ingo Molnar <mingo@...e.hu>
Date: Tue, 25 Nov 2008 18:14:38 +0100

>     fix warning in drivers/net/s2io.c
>     fix warnings in drivers/net/niu.c
>     fix warning in drivers/net/niu.c
>     fix warning in drivers/net/qla3xxx.c
>     fix warning in net/sunrpc/xprtrdma/svc_rdma_transport.c
>     fix warning in net/sched/sch_hfsc.c

Applied.

>     fix warning in net/netfilter/nf_conntrack_ftp.c

Patrick got this one.

>     fix warning in net/ipv6/ip6_flowlabel.c
>     fix warning in net/dsa/mv88e6xxx.c
>     fix warning in net/dsa/mv88e6060.c
>     fix warning in fs/dlm/netlink.c
>     fix warning in drivers/net/sis900.c

Applied

>     fix warning in drivers/net/wireless/iwlwifi/iwl-3945.c
>     fix warning in drivers/net/wireless/b43/main.c
>     fix warning in drivers/net/wireless/ath9k/rc.c

Needs to be submitted via wireless folks at linux-wireless

>     fix warning in drivers/net/wan/z85230.c
>     fix error path in drivers/net/mlx4/en_rx.c
>     fix warning in drivers/net/cassini.c
>     fix warning in net/dccp/options.c
>     fix warning in drivers/net/mlx4/mcg.c
>     fix warning in net/ax25/sysctl_net_ax25.c
>     fix warning in net/sunrpc/xprtrdma/verbs.c
>     fix warning in net/bluetooth/rfcomm/sock.c

Applied.

>     fix warning in drivers/net/wireless/ray_cs.c

Submit to wireless maintainer please.

>     fix warning in net/netfilter/nf_conntrack_proto_sctp.c
>     fix warning in net/netfilter/ipvs/ip_vs_ctl.c

Patrick got these.

>     fix warning in drivers/net/atlx/atl2.c

Applied.

>     fix warning in net/netfilter/nf_conntrack_proto_tcp.c

Patrick got it.

>     fix warning in drivers/net/wireless/iwlwifi/iwl-scan.c

Wireless submission please.

>     fix warning in drivers/net/depca.c
>     fix warning in drivers/net/3c523.c

Applied.

>     drivers/net/wireless/b43/phy_g.c: type check debug printouts

Submit to wireless folks please.

>     fix warning in drivers/net/sky2.c
>     fix warning in net/sunrpc/svcauth_unix.c
>     fix warning in security/selinux/netnode.c
>     fix warning in net/packet/af_packet.c
>     fix warning in net/rfkill/rfkill.c

Not applied.  Changing this from a BUG() to a panic() is wrong.
BUG() kills the current execution context, and allows the system
to potentially continue.  Panic() takes down the entire machine.

The !CONFIG_BUG BUG() definition should be marked in such a way
that gcc thinks it is noreturn.  Otherwise BUG() isn't really
BUG() anymore.  The other option is to never allow BUG to be
disabled or have it always evaluate to a simple bug trap.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/