Comment 7 for bug 610525

Revision history for this message
Mark Callaghan (mdcallag) wrote :

The warnings about null are from your patch

Your patch changed buf_block_get_frame from this:
#ifdef UNIV_DEBUG
/*********************************************************************//**
Gets a pointer to the memory frame of a block.
@return pointer to the frame */
UNIV_INLINE
buf_frame_t*
buf_block_get_frame(
/*================*/
        const buf_block_t* block) /*!< in: pointer to the control block */
        __attribute__((pure));
#else /* UNIV_DEBUG */
# define buf_block_get_frame(block) (block)->frame
#endif /* UNIV_DEBUG */

to this, check the else branch

#ifdef UNIV_DEBUG
 848 /*********************************************************************//**
 849 Gets a pointer to the memory frame of a block.
 850 @return pointer to the frame */
 851 UNIV_INLINE
 852 buf_frame_t*
 853 buf_block_get_frame(
 854 /*================*/
 855 const buf_block_t* block) /*!< in: pointer to the control block */
 856 __attribute__((pure));
 857 #else /* UNIV_DEBUG */
 858 # define buf_block_get_frame(block) (block ? (block)->frame : 0)
 859 #endif /* UNIV_DEBUG */