mudlet-makers team mailing list archive

[David <645129@xxxxxxxxxxxxxxxxxx>]

This is the latest HEAD version for linux, downloaded on 21 Sept. I am using
Qt 4.6.3. The function is "TBuffer::wrap( int startLine )", I don't want to
post the entire function because its quite long, but the line is about 3
from the end:

inline int TBuffer::wrap( int startLine)
{
    ...
    for( int i=0; i<tempList.size(); i++ )
    {
        if( tempList[i].size() < 1 )
        {
            lineBuffer.append( QString() );
        }
        else
        {
            lineBuffer.append( tempList[i] );
        }
        timeBuffer.append( timeList[i] );
        dirty.push_back( true );
        promptBuffer.push_back( promptList[i] );
    }
    return insertedLines > 0 ? insertedLines : 0;
}

wot is at wotmud.org:2222.  But I've since found the same problem at Sloth
and Aardmud (it seems to happen when logging in). I haven't tried anything
else yet.

On Wed, Sep 22, 2010 at 9:04 PM, Heiko <645129@xxxxxxxxxxxxxxxxxx>
wrote:

> Hi David,
>
> what version are you using? line 2255 in my version doesn't seem to be
> like yours. Can you quote the entire function plz?
> If you are on Linux you need to link Mudlet against the qt-4.6-sdk libs.
> What's the url and port of wot?
>
> Heiko
>
> On 09/22/2010 03:21 PM, Vadim Peretokin wrote:
> > Hm. Thank you. What is your Qt version?
> >
> > On 2010-09-22 7:45 AM, "David"<645129@xxxxxxxxxxxxxxxxxx>  wrote:
> >
> > Public bug reported:
> >
> > Mudlet crashes shortly after connecting to wotmud.org, with a QList
> index
> > out of range error.  I have traced it to line 2255 of TBuffer.cpp - it
> seems
> > that promptList is not necessarily the same size as tempList.  The error
> can
> > be removed by replacing that line with:
> >         if( promptList.size()>= 1 )
> >         {
> >             promptBuffer.push_back( promptList[i] );
> >         }
> >         else
> >         {
> >             promptBuffer.push_back( false );
> >         }
> >   but this might have ramifications of which I am not aware, so I do not
> want
> > to propose it as a solution.
> >
> > ** Affects: mudlet
> >      Importance: Undecided
> >          Status: New
> >
> > --
> > mudlet crashes with wormud.org
> > https://bugs.launchpad.net/bugs/645129
> > You received this bug notification because you are a member of Mudlet
> > Makers, which is subscribed to Mudlet.
> >
> > Status in Mudlet the MUD client: New
> >
> > Bug description:
> > Mudlet crashes shortly after connecting to wotmud.org, with a QList
> index
> > out of range error.  I have traced it to line 2255 of TBuffer.cpp - it
> seems
> > that promptList is not necessarily the same size as tempList.  The error
> can
> > be removed by replacing that line with:
> >         if( promptList.size()>= 1 )
> >         {
> >             promptBuffer.push_back( promptList[i] );
> >         }
> >         else
> >         {
> >             promptBuffer.push_back( false );
> >         }
> >   but this might have ramifications of which I am not aware, so I do not
> want
> > to propose it as a solution.
> >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~mudlet-makers<https://launchpad.net/%7Emudlet-makers>
> > Post to     : mudlet-makers@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~mudlet-makers<https://launchpad.net/%7Emudlet-makers>
> > More help   : https://help.launchpad.net/ListHelp
> >
> >
>
> --
> mudlet crashes with wormud.org
> https://bugs.launchpad.net/bugs/645129
> You received this bug notification because you are a direct subscriber
> of the bug.
>

-- 
mudlet crashes with wormud.org
https://bugs.launchpad.net/bugs/645129
You received this bug notification because you are a member of Mudlet
Makers, which is subscribed to Mudlet.

Status in Mudlet the MUD client: New

Bug description:
Mudlet crashes shortly after connecting to wotmud.org, with a QList index out of range error.  I have traced it to line 2255 of TBuffer.cpp - it seems that promptList is not necessarily the same size as tempList.  The error can be removed by replacing that line with:
	if( promptList.size() >= 1 )
	{
            promptBuffer.push_back( promptList[i] );
	}
	else
	{
	    promptBuffer.push_back( false );
	}
 but this might have ramifications of which I am not aware, so I do not want to propose it as a solution.