Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

no limitation for "%s" while calling fscanf() #110

Closed
QiAnXinCodeSafe opened this issue Dec 30, 2019 · 0 comments
Closed

no limitation for "%s" while calling fscanf() #110

QiAnXinCodeSafe opened this issue Dec 30, 2019 · 0 comments

Comments

@QiAnXinCodeSafe
Copy link

openbmc/common/recipes-core/fw-util/files/system.cpp

Line 82 in 6770592

if (fscanf(fp, "OpenBMC Release %s\n", vers) == 1) {

@QiAnXinCodeSafe QiAnXinCodeSafe changed the title Field width in scanf format string may case overflow no limitation for "%s" while calling fscanf() Dec 30, 2019
@leilanon leilanon mentioned this issue Apr 17, 2020
Closed
facebook-github-bot pushed a commit that referenced this issue Nov 19, 2020
@joancaneus @facebook-github-bot
ELBERT: dpm_ver.sh: Add SMB isl/raa versions (#110)
a28d62d 
Summary:
ELBERT: dpm_ver.sh: Add SMB isl/raa versions

Testing:

root@bmc-oob:/tmp/elbert_renasis_p1_onetimeupgrade# dpm_ver.sh
!!!
!!! WARNING: Displaying Digital Power Manager (DPM) version may disrupt sensor reading !!!
!!!
SCM.: SFT013030201
SMB.: SFT013180104
PIM2.: SFT013860103
PIM3.: SFT012990103
PIM4.: SFT013860103
PIM5.: SFT013860103
PIM6.: SFT012990103
PIM7.: SFT012990103
PIM8.: SFT012990103
PIM9.: SFT012990103
SMB ISL68226: SFT013200105
SMB RAA228228: SFT000000102
!!! Done.

Pull Request resolved: facebookexternal/openbmc.arista#110

Reviewed By: mikechoifb

fbshipit-source-id: 050d81d006
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fixes issue #110
1 participant
@QiAnXinCodeSafe