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.

Sign up for GitHub

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

Jump to bottom

Export bus array out of bounds causes crash on server #7312

Closed
Brunnn opened this issue Aug 3, 2023 · 10 comments · Fixed by #7313
Closed

Export bus array out of bounds causes crash on server #7312

Brunnn opened this issue Aug 3, 2023 · 10 comments · Fixed by #7313
Labels
bug Self explanatory?

Comments

@Brunnn
Copy link

Brunnn commented Aug 3, 2023

Describe the bug

Export bus is apparantly causing the server to crash.

It was working well for some time, but now it crashes everytime I open the server.

How to reproduce the bug

Not quite sure

Expected behavior

The export bus work like expected

Additional details

I was making some crafting automation using export bus, crafting units, etc..

Which minecraft version are you using?

1.20

On which mod loaders does it happen?

Forge

Crash log

https://pastebin.com/raw/J9Q2663N
@Brunnn Brunnn added the bug Self explanatory? label Aug 3, 2023
@Brunnn Brunnn changed the title Export bus array out of bounts causes crash on server Export bus array out of bounds causes crash on server Aug 3, 2023
@Brunnn
Copy link
Author

Brunnn commented Aug 3, 2023

Yep it was the export bus that was crashing the server.
In order to fix my world, i've got the block position in the crash-report
and ran the fill {{position}} diamond_block to remove it before the server crashed.

@shartte
Copy link
Member

shartte commented Aug 3, 2023

Yeah, I think that is a bug with the expanded filter size of the export bus in combination with a crafting upgrade card.

62832 added a commit that referenced this issue Aug 3, 2023
@62832
Update export bus crafting tracker to use new filter size
8fc24db 
Fixes #7312
@62832 62832 mentioned this issue Aug 3, 2023
Merged
shartte added a commit that referenced this issue Aug 3, 2023
@62832 @shartte
Fix #7312: Export bus with crafting card causes crash using slots bey…
06472a5 
…ond the first 9 (#7313)

Fixes #7312.

---------

Co-authored-by: Sebastian Hartte <shartte@users.noreply.github.com>
@Brunnn
Copy link
Author

Brunnn commented Aug 4, 2023

Are you guys planning a small release any time soon? would be great to use the export bus to automate things...

@62832
Copy link
Member

62832 commented Aug 4, 2023

There's already been a release made featuring this fix.

@Brunnn
Copy link
Author

Brunnn commented Aug 4, 2023

the last tag was for fabric, isn't it?

@shartte
Copy link
Member

shartte commented Aug 4, 2023

Yeah we.ran into trouble when trying to make the forge release due to other changes breaking the build and I am out of town until Tuesday. So no guarantees that the fix can be released for Forge before Tuesday

@shartte
Copy link
Member

shartte commented Aug 5, 2023

15.0.7-beta should be available on CF / Modrinth soon

@Brunnn
Copy link
Author

Brunnn commented Aug 6, 2023

Still not fixed in .8
see:
https://pastebin.com/npzatSKy

@shartte
Copy link
Member

shartte commented Aug 6, 2023

Urgh I think I made @62832 fix worse when I tried to improve it

@shartte shartte reopened this Aug 6, 2023
@shartte
Copy link
Member

shartte commented Aug 6, 2023

Urgh sorry, that's what I get for trying to release from out of town ;-)

The fixes were actually not included in 15.0.7/.8.
I just started the release of 15.0.9, which will include the missing fix for this.
It may take a few hours until it becomes visible on CF though.

@shartte shartte closed this as completed Aug 6, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Self explanatory?
Projects
None yet
Milestone
No milestone
3 participants
@shartte @62832 @Brunnn