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

after HD[X]9_imaged.hda is written the next file isn't writable #175

Closed
dericed opened this issue Aug 21, 2024 · 2 comments
Closed

after HD[X]9_imaged.hda is written the next file isn't writable #175

dericed opened this issue Aug 21, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@dericed
Copy link

dericed commented Aug 21, 2024

On my SD card, I have these files:

HD60_imaged.hda
HD61_imaged.hda
HD62_imaged.hda
HD63_imaged.hda
HD64_imaged.hda
HD65_imaged.hda
HD66_imaged.hda
HD67_imaged.hda
HD68_imaged.hda
HD69_imaged.hda
bluescsi.ini
log.txt

When I begin a subsequent transfer, I get these lines in the log:

[22595ms] Using filename: HD6:_imaged.hda to avoid overwriting existing file.
[22595ms] Failed to open file for writing: HD6:_imaged.hda

Perhaps the increment sets a : (colon) as the value after 9.

I can work around this by removing the existing files.

@erichelgeson erichelgeson added bug Something isn't working good first issue Good for newcomers labels Aug 21, 2024
@erichelgeson
Copy link
Contributor

Thanks for reporting! Yes it's due to me just simply incrementing the lun, which when it gets to 9 in ascii the next value is :. A simple printf or something would be better instead of modifying that one byte. https://github.com/BlueSCSI/BlueSCSI-v2/blob/main/src/BlueSCSI_initiator.cpp#L271

A good first issue for someone who might want to contribute.

@erichelgeson erichelgeson self-assigned this Sep 22, 2024
@erichelgeson erichelgeson removed the good first issue Good for newcomers label Sep 22, 2024
@dericed
Copy link
Author

dericed commented Sep 30, 2024

thank you! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants