Cache的Insert 和Add 方法引发的血案

版权声明:本文为博主原创文章,未经博主允许不得转载。 https://blog.csdn.net/moeryang/article/details/51133492

Cache的Insert 和Add 方法引发的血案

把一个简单的CMS系统拷贝到阿里云2008服务器上后,执行一个页面操作的时候,第一次操作正常,第二次或者第三次的时候,IIS就假死。查看日志:“**错误应用程序名称: w3wp.exe,版本: 7.5.7601.17514,时间戳: 0x4ce7a5f8
错误模块名称: KERNELBASE.dll,版本: 6.1.7601.19045,时间戳: 0x56258f05**”“错误存储段 ,类型 0 事件名称: APPCRASH 响应: 不可用”。

百度了一下,各种说法都有,有说权限不够的,有说程序有错误的。于是各种权限实验,还是报同样的错误。注意到其中有一条是说,程序里面有死循环所以导致溢出。联想到程序在2003下跑得欢快,而且每次都是点特定的页面第二三次,iis才crash。

开始测试数据库是否正常。自己写了和crash页面相同功能的操作,实验多次,发现不是数据库问题。那么应该是代码问题了。不过没有源码啊没有源码。没有办法只有反对编译,使用ILSpy 反编译代码。

看源代码好像没有什么问题啊,很简单,就是获取参数,然后塞入数据库。

没法子,把源代码拷贝出来,直接放在aspx运行,还是crash,把怀疑的注释掉,终于在我注释掉一个

Lib.Second.Tags.AddTagListToCache(this.siteId);

后IIS的日志里面再也没有crash了。再反编译这个方法其实是调用了一个Cache类,再反编译:

“`
public class NewsCache
{
private static object syncObj;
protected int _timeOut = 6000;
protected static volatile Cache webCache;
public int TimeOut
{
get
{
return (this._timeOut > 0) ? this._timeOut : 6000;
}
set
{
this._timeOut = ((value > 0) ? value : 6000);
}
}
public static Cache GetNewsCache
{
get
{
return NewsCache.webCache;
}
}
static NewsCache()
{
NewsCache.syncObj = new object();
NewsCache.webCache = null;
NewsCache.webCache = HttpRuntime.Cache;

        lock (NewsCache.syncObj)
        {
            HttpContext current = HttpContext.Current;
            if (current != null)
            {
                NewsCache.webCache = current.Cache;
            }
            else
            {
                NewsCache.webCache = HttpRuntime.Cache;
            }
        }
    }
    public void AddObject(string objId, object o)
    {
        if (objId != null && objId.Length != 0 && o != null)
        {
            CacheItemRemovedCallback cacheItemRemovedCallback = new CacheItemRemovedCallback(this.onRemove);
            if (this.TimeOut == 60)
            {
               // if (NewsCache.webCache[objId] == null)
                //{
                    NewsCache.webCache.Insert(objId, o, null, DateTime.MaxValue, TimeSpan.Zero, CacheItemPriority.High, cacheItemRemovedCallback);
                //}
            }
            else
            {

               // if (NewsCache.webCache[objId] == null)
               // {
                    NewsCache.webCache.Insert(objId, o, null, DateTime.Now.AddMinutes((double)this.TimeOut),
                     Cache.NoSlidingExpiration, CacheItemPriority.High, cacheItemRemovedCallback);
               // }
            }
        }
    }
    public void onRemove(string key, object val, CacheItemRemovedReason reason)
    {
        switch ((int)reason)
        {
            case 1:
                {
                    CacheItemRemovedCallback cacheItemRemovedCallback = new CacheItemRemovedCallback(this.onRemove);
                    NewsCache.webCache.Insert(key, val, null, DateTime.Now.AddMinutes((double)this.TimeOut),
                        Cache.NoSlidingExpiration, CacheItemPriority.High, cacheItemRemovedCallback);
                    break;
                }
            case 2:
                {
                    CacheItemRemovedCallback cacheItemRemovedCallback2 = new CacheItemRemovedCallback(this.onRemove);
                    NewsCache.webCache.Insert(key, val, null, DateTime.Now.AddMinutes((double)this.TimeOut),
                        Cache.NoSlidingExpiration, CacheItemPriority.High, cacheItemRemovedCallback2);
                    break;
                }
            case 3:
                {
                    CacheItemRemovedCallback cacheItemRemovedCallback3 = new CacheItemRemovedCallback(this.onRemove);
                    NewsCache.webCache.Insert(key, val, null, DateTime.Now.AddMinutes((double)this.TimeOut),
                        Cache.NoSlidingExpiration, CacheItemPriority.High, cacheItemRemovedCallback3);
                    break;
                }
        }
    }
    public void RemoveObject(string objId)
    {
        if (objId != null && objId.Length != 0)
        {
            NewsCache.webCache.Remove(objId);
        }
    }
    public object RetrieveObject(string objId)
    {
        object result;
        if (objId == null || objId.Length == 0)
        {
            result = null;
        }
        else
        {
            result = NewsCache.webCache.Get(objId);
        }
        return result;
    }
}

值得注意的是这个Cache类Insert的时候传入的一个回掉事件,在回掉事件里面又把Cache的数据赛进去了,大概是想这个Cache永远不过期吧。

是的这个类AddObject 方法如果你多次叫用之后IIS就会挂掉,onRemove 会不挺的被调用。换句说就是对于Cache来说如果你多次调用Insert方法,而且Insert后面有回掉事件的话,这个事件就会被执行。之前的程序员在onRemove 里面又Insert了一次,于是这个onRemove 就死循环了。奇怪的是这个问题在IIS 6上并不会发生或者说在win 2003 server的IIS 6 的.Net 2.0 上并不会发生。在IIS 7 或者 7.5 的 .Net 2.0 上会发生。

当然解决的办法是把Insert换成Add,或者在Insert的时候判断一次这个Cache是否为Null。

扫描二维码关注公众号,回复: 5086198 查看本文章

**百度上解释的Cache 的Insert 方法是先移除再添加数据,不过msdn上没有提到insert多次会触发回掉事件。
Add 是如果有同名的键会有异常,实验证明Add有同名的也不会报错。**

最后:

ILSpy真是个好东西,反编译很棒,还免费的
Aspx文件去掉 Page language=”c#” Codebehind=”YouClass.aspx.cs” YouClass就可以把代码写到aspx文件里面动态编译,没有源代码,只要不太复制也可以搞定。

猜你喜欢

转载自blog.csdn.net/moeryang/article/details/51133492