Skip to content

Commit d40a7e6

Browse files
committed
common/ompio: fix division by zero problem with empty fview
When using an empty fileview, a division by zero bug can occur in ompio. Not entirely sure why the problem did not show up previously, but some recent changes trigger that bug in one of our tests. This pr is part of a fix applied in commit f6b3a0a Fixes Issue #6703 Signed-off-by: Edgar Gabriel <[email protected]>
1 parent a06ad1a commit d40a7e6

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

ompi/mca/common/ompio/common_ompio_file_open.c

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99
* University of Stuttgart. All rights reserved.
1010
* Copyright (c) 2004-2005 The Regents of the University of California.
1111
* All rights reserved.
12-
* Copyright (c) 2008-2017 University of Houston. All rights reserved.
13-
* Copyright (c) 2015-2017 Research Organization for Information Science
12+
* Copyright (c) 2008-2019 University of Houston. All rights reserved.
13+
* Copyright (c) 2015-2018 Research Organization for Information Science
1414
* and Technology (RIST). All rights reserved.
1515
* Copyright (c) 2016 Cisco Systems, Inc. All rights reserved.
1616
* $COPYRIGHT$
@@ -378,6 +378,13 @@ int mca_common_ompio_file_get_position (mca_io_ompio_file_t *fh,
378378
{
379379
OMPI_MPI_OFFSET_TYPE off;
380380

381+
if ( 0 == fh->f_view_extent ||
382+
0 == fh->f_view_size ||
383+
0 == fh->f_etype_size ) {
384+
*offset = 0;
385+
return OMPI_SUCCESS;
386+
}
387+
381388
/* No. of copies of the entire file view */
382389
off = (fh->f_offset - fh->f_disp)/fh->f_view_extent;
383390

0 commit comments

Comments
 (0)