Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WCCMMUD] Majormud - Game sysop's WCCMMUD.INI full screen editor crashes at 80 characters #545

Open
fletcherm opened this issue Jan 20, 2022 · 1 comment
Labels
bug Something isn't working full screen display aka full screen editor majormud

Comments

@fletcherm
Copy link
Member

fletcherm commented Jan 20, 2022

Module Information

  • Identifier WCCMMUD
  • Module Name MajorMud
  • Version 1.11p

Describe the bug

Using the full screen editor to edit WCCMMUD.INI (in the sysop menu) crashes the system if the file is > 79 characters.

To Reproduce
Steps to reproduce the behavior:

  1. Ensure your WCCMMUD.INI file on disk is > 79 characters.
  2. Ensure your BBS user has the WCCSYSOP key - this grants access to Majormud's Sysop menu
  3. Enter MajorMud and type 'sysop' to enter the sysop menu
  4. Type E to edit the WCCMMUD.INI file
  5. Full screen editor will briefly show, it will look all messed up, and the board will crash with this stacktrace in the background:
mbbsemu    | Unhandled exception. System.InvalidOperationException: Sequence contains no matching element
mbbsemu    |    at System.Linq.ThrowHelper.ThrowNoMatchException()
mbbsemu    |    at MBBSEmu.HostProcess.Fsd.FsdUtility.SetAnswers(List`1 answers, List`1 fields)
mbbsemu    |    at MBBSEmu.HostProcess.HostRoutines.FsdRoutines.EnteringFullScreenDisplay(SessionBase session)
mbbsemu    |    at MBBSEmu.HostProcess.HostRoutines.FsdRoutines.ProcessSessionState(SessionBase session, Dictionary`2 modules)
mbbsemu    |    at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
mbbsemu    |    at System.Threading.Thread.StartCallback()

Through trial and error I found that files <= 79 don't crash, but > 79 character do.

For example: this file content (79 characters) does not crash:

# GENERAL
BLANK_LASTNAME_ALLOWED=1
CODEC_OF_TWINKS=78
MAXIMUM_BANK_BALANCE=0
B

And this content (80 characters) does crash:

# GENERAL
BLANK_LASTNAME_ALLOWED=1
CODEC_OF_TWINKS=78
MAXIMUM_BANK_BALANCE=0
BO

Expected behavior

The board does not crash for content > 79 characters.

Software Information:

  • OS: Ubuntu 20.04, Docker, mbbsemu-docker image
  • Telnet Client Megamud
@fletcherm
Copy link
Member Author

@synacktic I'd appreciate if you could try this with your WG reference system. Mine is all messed up right now.

The game menu does warn "only [edit] if [this file] is small," so being too long could be a known problem...but a hard hard hard crash like this, with that stacktrace, would surprise me.

Would be interesting to see what you get on a reference system.

@fletcherm fletcherm added the bug Something isn't working label Jan 21, 2022
@fletcherm fletcherm changed the title [WCCMMUD] Majormud - Sysop's WCCMMUD.INI full screen editor crashes at 80 characters [WCCMMUD] Majormud - Game sysop's WCCMMUD.INI full screen editor crashes at 80 characters Jan 23, 2022
@enusbaum enusbaum added the module Defect/Issue related to a specific Module label Jan 25, 2022
@fletcherm fletcherm added majormud and removed module Defect/Issue related to a specific Module labels Mar 16, 2022
@fletcherm fletcherm added the full screen display aka full screen editor label Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working full screen display aka full screen editor majormud
Projects
None yet
Development

No branches or pull requests

2 participants