Message ID | 1421175905-17035-9-git-send-email-colin.king@canonical.com |
---|---|
State | Accepted |
Headers | show |
On 01/14/2015 03:04 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > cppcheck is picking up some minor style issues which can > be easily fixed: > > [src/uefi/uefirtvariable/uefirtvariable.c:254]: > (style) The scope of the variable 'i' can be reduced. > [src/uefi/uefirtvariable/uefirtvariable.c:1200]: > (style) The scope of the variable 'ret' can be reduced. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > src/uefi/uefirtvariable/uefirtvariable.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/uefi/uefirtvariable/uefirtvariable.c b/src/uefi/uefirtvariable/uefirtvariable.c > index 94caf0d..0c3f532 100644 > --- a/src/uefi/uefirtvariable/uefirtvariable.c > +++ b/src/uefi/uefirtvariable/uefirtvariable.c > @@ -251,13 +251,14 @@ err_restore_env: > static bool compare_guid(const EFI_GUID *guid1, const EFI_GUID *guid2) > { > bool ident = true; > - int i; > > if ((guid1->Data1 != guid2->Data1) || > (guid1->Data2 != guid2->Data2) || > (guid1->Data3 != guid2->Data3)) > ident = false; > else { > + int i; > + > for (i = 0; i < 8; i++) { > if (guid1->Data4[i] != guid2->Data4[i]) > ident = false; > @@ -1197,7 +1198,6 @@ static int setvariable_test5(fwts_framework *fw) > > static int setvariable_test6(fwts_framework *fw) > { > - int ret; > uint64_t datasize = 10; > uint8_t datadiff = 0; > uint32_t attributesarray[] = { > @@ -1208,7 +1208,7 @@ static int setvariable_test6(fwts_framework *fw) > uint64_t index; > > for (index = 0; index < (sizeof(attributesarray)/(sizeof attributesarray[0])); index++) { > - ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); > + int ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); > > if (ret == FWTS_ERROR) { > /* successfully set variable with invalid attributes, test fail */ > Acked-by: Ivan Hu <ivan.hu@canonical.com>
On Wed, Jan 14, 2015 at 3:04 AM, Colin King <colin.king@canonical.com> wrote: > From: Colin Ian King <colin.king@canonical.com> > > cppcheck is picking up some minor style issues which can > be easily fixed: > > [src/uefi/uefirtvariable/uefirtvariable.c:254]: > (style) The scope of the variable 'i' can be reduced. > [src/uefi/uefirtvariable/uefirtvariable.c:1200]: > (style) The scope of the variable 'ret' can be reduced. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > src/uefi/uefirtvariable/uefirtvariable.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/uefi/uefirtvariable/uefirtvariable.c b/src/uefi/uefirtvariable/uefirtvariable.c > index 94caf0d..0c3f532 100644 > --- a/src/uefi/uefirtvariable/uefirtvariable.c > +++ b/src/uefi/uefirtvariable/uefirtvariable.c > @@ -251,13 +251,14 @@ err_restore_env: > static bool compare_guid(const EFI_GUID *guid1, const EFI_GUID *guid2) > { > bool ident = true; > - int i; > > if ((guid1->Data1 != guid2->Data1) || > (guid1->Data2 != guid2->Data2) || > (guid1->Data3 != guid2->Data3)) > ident = false; > else { > + int i; > + > for (i = 0; i < 8; i++) { > if (guid1->Data4[i] != guid2->Data4[i]) > ident = false; > @@ -1197,7 +1198,6 @@ static int setvariable_test5(fwts_framework *fw) > > static int setvariable_test6(fwts_framework *fw) > { > - int ret; > uint64_t datasize = 10; > uint8_t datadiff = 0; > uint32_t attributesarray[] = { > @@ -1208,7 +1208,7 @@ static int setvariable_test6(fwts_framework *fw) > uint64_t index; > > for (index = 0; index < (sizeof(attributesarray)/(sizeof attributesarray[0])); index++) { > - ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); > + int ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); > > if (ret == FWTS_ERROR) { > /* successfully set variable with invalid attributes, test fail */ > -- > 2.1.4 > > Acked-by: Keng-Yu Lin <kengyu@canonical.com>
diff --git a/src/uefi/uefirtvariable/uefirtvariable.c b/src/uefi/uefirtvariable/uefirtvariable.c index 94caf0d..0c3f532 100644 --- a/src/uefi/uefirtvariable/uefirtvariable.c +++ b/src/uefi/uefirtvariable/uefirtvariable.c @@ -251,13 +251,14 @@ err_restore_env: static bool compare_guid(const EFI_GUID *guid1, const EFI_GUID *guid2) { bool ident = true; - int i; if ((guid1->Data1 != guid2->Data1) || (guid1->Data2 != guid2->Data2) || (guid1->Data3 != guid2->Data3)) ident = false; else { + int i; + for (i = 0; i < 8; i++) { if (guid1->Data4[i] != guid2->Data4[i]) ident = false; @@ -1197,7 +1198,6 @@ static int setvariable_test5(fwts_framework *fw) static int setvariable_test6(fwts_framework *fw) { - int ret; uint64_t datasize = 10; uint8_t datadiff = 0; uint32_t attributesarray[] = { @@ -1208,7 +1208,7 @@ static int setvariable_test6(fwts_framework *fw) uint64_t index; for (index = 0; index < (sizeof(attributesarray)/(sizeof attributesarray[0])); index++) { - ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); + int ret = setvariable_invalidattr(fw, attributesarray[index], datasize, variablenametest, >estguid1, datadiff); if (ret == FWTS_ERROR) { /* successfully set variable with invalid attributes, test fail */