Some missing splnet() in key.c

From: VANHULLEBUS Yvan (yvan.vanhullebus_at_netasq.com)
Date: 08/17/05

  • Next message: VANHULLEBUS Yvan: "Re: Some missing splnet() in key.c"
    Date: Wed, 17 Aug 2005 20:23:49 +0200
    To: freebsd-net@freebsd.org
    
    
    
    

    Hi all.

    A few months ago, I reported some missing splnet() in key.c to
    snap-users@kame.net. I found them by tracking some random and strange
    problems, which are more likely to happen when running on a "slow"
    CPU, when having some heavy PFKey activity and when having high IPSec
    traffic.

    The attached patch (made against FreeBSD6 version, but should be easy
    to port to other versions) fixes at least most splnet problems (well,
    at least, I didn't have any more report for customers which use the
    latest version including all those locks....).

    Please note that mixing this patch and the FreeBSD NAT-T patch
    available on ipsec-tools web site will have a possible dead lock in
    key_add(), when handling NAT-T extensions (Manu: check that for
    NetBSD, there is probably the same code !).

    I'll update quickly FreeBSD6 NAT-T patchset on ipsec-tools web site if
    this patch is commited on FreeBSD6 source.

    Yvan.

    -- 
    NETASQ - Secure Internet Connectivity
    http://www.netasq.com
    
    



  • Next message: VANHULLEBUS Yvan: "Re: Some missing splnet() in key.c"

    Relevant Pages

    • SUMMARY: Solaris Patching
      ... a bunch of people reported annoyances uses patchpro... ... Use the pprosetup command to set parameters (pprosetup -h, ... against SUN's patch database. ... i use sun's patchk to report outstanding ...
      (SunManagers)
    • Re: Patch 2.4.3 PTR Patch Notes
      ... I'll likely stick with travelform till the patch given the cheapness of ... drop rate of blue gems from mining nodes remains unchanged. ... no longer be required to select a category when opening a GM ticket. ... that requires a GM response or the ?Report Issue? ...
      (alt.games.warcraft)
    • Re: Linux-2.6.13-rc7
      ... > report in first, to verify if this patch works as advertised. ... m68k (and applies both to Linus' tree and m68k CVS); ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: KB891711 & Final Fantasy XI Blue-Screen
      ... you can uninstall KB891711 - although it's not recommended. ... Use Add/Remove Programs for that. ... MS are actively looking at the problem with the patch - hopefully they'll ... report. ...
      (microsoft.public.windowsupdate)
    • Re: Giving developers clue how many testers verified certain kernel version
      ... I do report bugs and will always ... no problem when people do NOT report bugs in that piece of code ... patch to the official tree. ... > in your ACPI example there is a bug/problem. ...
      (Linux-Kernel)