Skip to content

Commit 955d5c9

Browse files
committed
Correct the types of some error returns
1 parent 2ea0713 commit 955d5c9

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/dicom-file.c

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@ static DcmDataSet *dcm_filehandle_read_file_meta(DcmError **error,
536536
false,
537537
&parse,
538538
filehandle)) {
539-
return false;
539+
return NULL;
540540
}
541541

542542
/* Sanity check. We should have parsed a single dataset into the sequence
@@ -555,8 +555,8 @@ static DcmDataSet *dcm_filehandle_read_file_meta(DcmError **error,
555555
}
556556

557557
DcmDataSet *file_meta = dcm_sequence_get(error, sequence, 0);
558-
if (file_meta == NULL ) {
559-
return false;
558+
if (file_meta == NULL) {
559+
return NULL;
560560
}
561561

562562
// steal file_meta to stop it being destroyed
@@ -765,7 +765,7 @@ DcmDataSet *dcm_filehandle_read_metadata(DcmError **error,
765765
filehandle->implicit,
766766
&parse,
767767
filehandle)) {
768-
return false;
768+
return NULL;
769769
}
770770

771771
/* Sanity check. We should have parsed a single dataset into the sequence
@@ -784,8 +784,8 @@ DcmDataSet *dcm_filehandle_read_metadata(DcmError **error,
784784
}
785785

786786
DcmDataSet *meta = dcm_sequence_get(error, sequence, 0);
787-
if (meta == NULL ) {
788-
return false;
787+
if (meta == NULL) {
788+
return NULL;
789789
}
790790

791791
// steal meta to stop it being destroyed
@@ -838,7 +838,7 @@ const DcmDataSet *dcm_filehandle_get_metadata_subset(DcmError **error,
838838
&filehandle->tiles_across, &filehandle->tiles_down) ||
839839
!set_pixel_description(error, meta, &filehandle->desc)) {
840840
dcm_dataset_destroy(meta);
841-
return false;
841+
return NULL;
842842
}
843843
filehandle->num_tiles = filehandle->tiles_across *
844844
filehandle->tiles_down;

0 commit comments

Comments
 (0)