cache_get flawed

I've tracked down a bug, don't know for sure but seems related to external cache (memcache).

The module was not working properly, data would go into the database, but no changes were visible.

The issue is in line 17 of msnf.steps.inc

 if (($cached = cache_get('msnf_step_data:'. $language->language)) !== 0) { 

Seems like it's caused by cache_get returning expired cached items, as seen here:

http://api.drupal.org/api/drupal/includes%21cache.inc/function/cache_get...

Changing line 17 to

 if (($cached = cache_get('msnf_step_data:'. $language->language)) !== 0 && !empty($cached->data) && (time() < $cached->expire)) {

as suggested in the above link seems to fix the issue.