Might be coincidence but the Brazilian QTV seems to be working fine now.

Unfortunately the butt.se QTV seems to be dead, it was covering some Russian servers.

Still no more clues as to when this bug occurs? Some servers look so broken on QTV sometimes.

Not just qtv, but http://quakeservers.net. Often just the players / spectators sections.

I restarted the Besmella QTV a few days ago. Thanks to the 3 people who let me know it was fucked.

As I am not so active anymore, if it needs to be done again then you can email me dirtbox {at} besmella [dot] com.

Watching a mix on quakeworld.co.uk:27501 at the moment and two of the players are completely missing from the scoreboard in game and QTV page shows empty blank names, confusing...

Edit: and they're back when game was finished and map reloaded...

No brilliant ideas on how to fix these problems? :|

Right now nl.besmella.com:28501 is bugged on http://qtv.quakeworld.nu/ and in-game. One player completely gone from scoreboard.

This REALLY needs fixing!

yep this is really annoying.
The mvdsv, ktx and qtv binaries are all up to date on quakeworld.co.uk and it still happens, completely randomly. Reloading the map will (sometimes) fix it, restarting qtv will (sometimes) fix it.
Time to start sifting through the qtv code I guess :<

Believe this is caused by the userinfo for the player being corrupted. Example from bugged qtv connection yesterday:

\*client\ezQuake 4306\ns\toposmos\topcolor\13\bottomcolor\13\team\blue

No \name\ key => blank string in QTV, topcolor/bottomcolor being corrupt => white colours in QTV, *spectator being corrupt => spec appearing as player.
Restarting the map should temporarily fix this, as the full userinfo key is sent again rather than individual key/value updates.

Replacing the call to strcpy with Q_strcpy from ezquake fixed this for me in very limited testing. strcpy has undefined behaviour for overlapping strings, if it copies from the start of the string then the bug shouldn't appear (I couldn't reproduce bug on Windows).

Superb! Thanks a million for this Meag. It's great to have an experienced coder look at this, rather than me floundering around in the code

I'll apply your fix when I get home and see how we get on.
Welcome back btw

Nice one.

epic stuff meag!

modified version is up and running on quakeworld.co.uk

Thanks for this - have connected on and haven't been able to make the bug happen. On besmella doing something as simple as alt-tabbing (which sets the chat userinfo key) can make it start to show incorrect player info.

There might be other reasons this is happening so we'll have to keep an eye on it, but looks good so far.

Can someone give this fix to dirtbox to fix nl.besmella? And nquake too?

yep. I've compiled this for 32bit and 64bit linux:

http://froschroom.com/qtv_linux_bin.zip

thx!
I will update nQuakesv with these.

EDIT: updated nQuake. Use fnu mirror if you download today

How are the binaries built, on which platform? Please take a patch instead and rebuild on the regular build machine.

You usually don't do binary distribution on Linux due to library incompatibilities (there are a bunch of different distributions with their own schedule). If you have to, then use Debian stable (not necessarily the latest) to build binaries, it's probably the best chance for highest compatibility.

I wouldn't normally do binary distribution either. Both the above were built on Debian stable for mushi. Once we've fully established it's fully fixed it would be very useful to integrate meag's change into your repo if you're ok with it.

I submitted patch here, is that where the latest qtv source is? Deurk seems to have commented on dimman's QTV repo that that is where the 'official' code is, so I went with that.

Also: I think this might also need fixed in mvdsv... I got deurk's mvdsv, patched it for ktx, then managed to replicate the same bug but in the serverinfo string (the server will start complaining that the serverinfo string is too long to add new keys - it has corrupted and so can't remove keys to make space for new values).

But I don't really know where to submit a patch for that, because deurk's mvdsv is not compatible with ktx (it has hifi's fix for standard mods) and he's refusing to update it until that is resolved so it runs both PR1 mods and KTX. Is everyone using deurk's + akat1's pull request in the meantime? I'm a bit confused.

Bump - can this please be applied to Besmella servers?

Done! I wish someone emailed me about this! I only stumbled upon this thread now...

Just saw the same bug on fvck.se:28502 - does anyone know who's admin and could update them with the fix?

*admin delete manellermus[at]gmail.com

from quakeservers.net

ran ./update_binaries.sh, dno if it helped.. let me know.